Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

2-6 [배포] [fix] actions 마지막 단계 미실행 현상 해결 #49

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

leesungbin
Copy link
Member

개요

pull_from_registry의 실행 순서는 push_to_registry_bepush_to_registry_fe 가 모두 실행된 후여야합니다.
하지만, 두 job 중 하나라도 실행되었다면 pull_from_registry는 실행되어야합니다.
현재는 backend 폴더와 frontend 폴더가 모두 수정되어야 작동됩니다.

작업사항

  • 2-6 [배포] [fix] checkout step 추가 #46 으로 인한 배포 자동화 막힘 현상 해결
  • PR이 머지된 후에는 아래 사진과 같이, 2단계에서의 2개의 job 중 하나라도 완료되면 서버로 배포를 진행합니다.
    image

리뷰 요청사항

  • N/A

pull_from_registry의 실행 순서는 push_to_registry_be와 push_to_registry_fe 가 모두 실행된 후여야한다.
하지만, 두 job 중 하나라도 실행되었다면 pull_from_registry는 실행되어야한다.
@leesungbin leesungbin marked this pull request as ready for review November 22, 2022 05:54
@github-actions github-actions bot requested review from JunYupK and Palwol November 22, 2022 05:54
@leesungbin leesungbin linked an issue Nov 22, 2022 that may be closed by this pull request
Copy link
Collaborator

@JunYupK JunYupK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

확인했습니다.

Copy link
Collaborator

@Palwol Palwol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

좋습니다.

@leesungbin leesungbin merged commit 901c4dc into dev Nov 22, 2022
@leesungbin leesungbin deleted the hotfix/auto-deploy branch November 22, 2022 06:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] github actions 자동배포 수정필요
3 participants