Skip to content
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.

Sign up for GitHub

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

Build: #BBB-159 환경변수로 CD workflow 활성화·비활성화 관리 #42

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Build: #BBB-159 환경변수로 CD workflow 활성화·비활성화 관리 #42

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .github/workflows/deploy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,8 @@ on:

jobs:
deploy:
if: ${{ secrets.ENABLE_CD == 'true' }}

env:
AWS_REGION: ap-northeast-2
ECR_REPOSITORY: devs-nextjs
Copy link

Choose a reason for hiding this comment

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

이 코드 패치에 대한 간단한 코드 리뷰를 진행하겠습니다.

코드 리뷰

  1. 조건문 검토:

    • if: ${{ secrets.ENABLE_CD == 'true' }} 조건문이 추가되었습니다. 이 조건문은 ENABLE_CD 비밀(Secrets) 환경 변수가 'true'일 경우에만 deploy 작업이 실행된다는 것을 의미합니다. 이 방식은 유용하지만, 만약 ENABLE_CD가 설정되지 않았을 경우 어떻게 되는지에 대한 처리가 필요해 보입니다. 기본값을 설정하거나 예외 처리를 고려할 수 있습니다.

  2. 비밀 관리:

    • GitHub의 비밀 환경변수는 안전하게 관리되지만, ENABLE_CD와 같은 설정을 사용하는 경우, 해당 값이 실제로 어떤 환경에서 사용될 것인지에 대해 문서화를 해두는 것이 좋습니다. 다른 협업자들이 이 설정의 목적을 이해할 수 있도록 주석을 추가하는 것이 도움이 될 것입니다.

  3. 환경 변수 관리:

    • AWS_REGION 및 ECR_REPOSITORY 환경 변수가 하드코딩 되어 있습니다. 이러한 값들도 비밀 환경변수로 관리하면 보안성이 향상될 수 있습니다. 예를 들어, 다른 환경 (개발, 테스트, 운영)에 따라 값이 달라질 수 있기 때문에 이를 유동적으로 관리하는 방법을 고려해보세요.

  4. 직관성:

    • 조건문 사용에 있어 'true'가 아닌 불리언으로 설정이 가능하다면, if: ${{ secrets.ENABLE_CD }} 형태로 변경하는 것도 고려해볼 수 있습니다. 이는 좀 더 직관적이어서 코드의 가독성을 높일 수 있습니다.

  5. 테스트:

    • 이 패치를 적용한 후, 조건문이 올바로 작동하는지에 대한 테스트가 필요합니다. 예를 들어, 실제로 ENABLE_CD 값이 'true'일 때와 아닐 때 배포 작업이 어떻게 수행되는지를 확인해야 합니다.

추가적인 개선 사항이나 필요 시 기능을 확장하는 것도 고려해보세요. 전체적으로 간단하고 유용한 패치입니다.

Expand Down
Loading