refactor: NCP server로 rollback#151
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the production Docker Compose configuration by removing the public port mapping for the MySQL database and switching the application image registry from GCP to NCR. Feedback was provided regarding the use of the "latest" tag in a production environment, which can lead to deployment instability, and the hardcoding of the registry URL instead of using environment variables.
|
|
||
| app: | ||
| image: ${GCP_REGION}-docker.pkg.dev/${GCP_PROJECT_ID}/${GCP_REPOSITORY}/link-it-backend:latest | ||
| image: link-it-registry.kr.ncr.ntruss.com/link-it-backend:latest |
There was a problem hiding this comment.
운영 환경(production)에서는 컨테이너 이미지에 latest 태그를 사용하는 것을 지양해야 합니다. latest 태그는 배포 시점에 어떤 버전의 이미지가 사용되는지 불분명하게 만들며, 이미지 업데이트 시 의도치 않은 변경이 발생할 수 있어 롤백과 디버깅을 어렵게 합니다. 특정 버전 태그(예: 커밋 해시 또는 시맨틱 버저닝)를 사용하는 것을 권장합니다.\n\n또한, 레지스트리 주소가 하드코딩되어 있습니다. 리포지토리 규칙에 따라 환경별 설정은 YAML 파일 내에서 기본값을 포함한 환경 변수를 사용하여 관리하는 것이 좋습니다.
References
- For environment-specific configurations in YAML files, use environment variables with default values instead of hardcoding or commenting out values.
1 participant
🔥ISSUE
NCP server로 rollback