-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CI CD 파이프라인이 눈에 잘 보이도록 스크립트 개선(Issue #88) #89
Conversation
여러 워크 플로우를 하나의 스크립트로 통합.
음 지나가는 길인데용. |
id: deploy | ||
run: ls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
아직 배포 스크립트를 작성하지 않아서 이대로 뒀어요! 이번 이슈의 목표는 어디까지나 워크플로우의 가시성 향상이니까요!
outputs: | ||
build: ${{ steps.build_result.outputs.result }} | ||
deploy: ${{ steps.deploy_result.outputs.result }} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
어떤 job 이나 step 의 결과를 저장하여 다른 step 이나 job에서 사용하고 싶은 경우 outputs 라는 컨텍스트를 사용하면 된다고 해서 적용했어요!
https://docs.github.com/ko/actions/using-jobs/defining-outputs-for-jobs
runs-on: ubuntu-latest | ||
needs: [CD] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
need 를 통해 job 사이에 의존성을 설정할 수 있어서 사용했어요! 의존을 설정하지 않으면 여러 job 은 병렬로 실행되지만, 의존성이 있다면 해당 부분을 순서대로 실행하게 할 수 있어요!
알림을 보내는 부분은 self-hosted
를 사용할 이유가 없어서 우분투에서 돌아가도록 했어요. 깃허브에서 이런식으로 job으로 구분하면, 두 job 사이에 지연이 발생한다는 점이 단점인 것 같아요!
여러 워크 플로우를 하나의 스크립트로 통합.
구현 요약
연관 이슈
close #88
참고
코드 리뷰에
RCA 룰
을 적용할 시 참고해주세요.