Conversation
Merge branch 'dev' into fix(#7)ci-cd-fix # Conflicts: # .github/workflows/cd-prod.yml # src/main/java/com/example/RealMatch/global/config/SecurityConfig.java
Summary of ChangesHello @ParkJiYeoung8297, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 PR은 지속적인 배포(CD) 환경을 개선하고, 서버 배포 시 발생했던 포트 접근 문제를 해결하며, 더 이상 필요 없는 임시 채팅 기능 관련 코드를 정리하는 것을 목표로 합니다. Highlights
Ignored Files
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
안녕하세요. CD 파이프라인을 이미지 pull 방식으로 변경하는 작업을 잘 확인했습니다. docker-compose.yaml 파일의 수정은 PR의 설명과 일치하며, restart: always 정책 추가와 포트 변경 모두 타당해 보입니다. 리뷰어에게 남겨주신 질문에 대해 답변 드리자면, .env와 docker-compose.yaml 파일을 서버에서 수동으로 관리하는 방식은 초기에는 간단할 수 있지만, 장기적으로는 저장소의 설정과 서버의 설정이 달라질 위험이 있고, 변경 사항을 수동으로 적용해야 하는 번거로움이 있습니다. GitHub Actions의 secrets 등을 활용해 배포 과정을 자동화하고 일관성을 유지하는 방식을 고려해보시는 것을 추천합니다. 추가로 docker-compose.yaml 파일에 대한 작은 개선 제안을 리뷰 댓글로 남겼으니 확인 부탁드립니다.
| build: | ||
| context: . | ||
| dockerfile: Docker/Dockerfile | ||
| image: ${DOCKERHUB_USERNAME}/realmatch-backend:prod |
There was a problem hiding this comment.
Docker 이미지 태그로 prod를 사용하고 계십니다. 이 방식은 새로운 이미지를 배포할 때마다 같은 태그를 덮어쓰게 되어, 특정 버전으로 롤백하기 어렵게 만들 수 있습니다.
Git commit hash나 시맨틱 버전(예: v1.2.3)과 같이 각 빌드를 고유하게 식별할 수 있는 태그를 사용하는 것을 권장합니다. 이렇게 하면 배포된 버전을 명확하게 추적하고, 문제 발생 시 안정적인 이전 버전으로 쉽게 롤백할 수 있습니다.
예시:
image: ${DOCKERHUB_USERNAME}/realmatch-backend:${IMAGE_TAG:-prod}CI/CD 파이프라인에서 IMAGE_TAG 환경 변수를 설정하여 동적으로 태그를 주입할 수 있습니다.
image: ${DOCKERHUB_USERNAME}/realmatch-backend:${IMAGE_TAG:-prod}
related_to : #7
📝 작업 내용
CD에서 image pull 방식으로 수정
✅ 변경 사항
📷 스크린샷 (선택)
💬 리뷰어에게