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

[platinouss]

작업 개요

전달 사항

Github Secrets에 저장된 boolean 값을 바탕으로 CD workflow의 활성화 또는 비활성화 적용

참고 자료

[github-actions]

Code review by ChatGPT

[github-actions]

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

코드 리뷰

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'일 때와 아닐 때 배포 작업이 어떻게 수행되는지를 확인해야 합니다.

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