Skip to content

Conversation

@LeeShinHaeng
Copy link
Contributor

@LeeShinHaeng LeeShinHaeng commented Dec 17, 2025

📋 상세 설명

  • 실제 운영서버에 배포를 위한 CD 파일을 추가했습니다.
  • 사용하지 않는 local-run 폴더의 docker-compose를 삭제했습니다.
  • application.yml 파일에 depreacted 속성을 최신화 했습니다.

  • 추가로 배포 시 필요한 docker-compose.yml과 환경변수는 배포 서버에 저장해 놓았습니다.

📊 체크리스트

  • PR 제목이 형식에 맞나요 e.g. feat: PR을 등록한다
  • 코드가 테스트 되었나요
  • 문서는 업데이트 되었나요
  • 불필요한 코드를 제거했나요
  • 이슈와 라벨이 등록되었나요

📆 마감일

Close #136

@LeeShinHaeng LeeShinHaeng self-assigned this Dec 17, 2025
@LeeShinHaeng LeeShinHaeng added the 🗂️chore 간단한 업무 label Dec 17, 2025
@LeeShinHaeng LeeShinHaeng linked an issue Dec 17, 2025 that may be closed by this pull request
2 tasks
@github-actions
Copy link

github-actions bot commented Dec 17, 2025

Unit Test Results

  36 files  ±0    36 suites  ±0   3s ⏱️ -1s
109 tests ±0  109 ✔️ ±0  0 💤 ±0  0 ±0 
110 runs  ±0  110 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 9d924e2. ± Comparison against base commit af8bc5f.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Dec 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@LeeShinHaeng LeeShinHaeng marked this pull request as ready for review December 17, 2025 13:56
@bsko98
Copy link
Collaborator

bsko98 commented Dec 25, 2025

사용하지 않는 local-run 폴더의 docker-compose를 삭제했습니다.

혹시 이 폴더에 local용 docker-compose를 이 폴더에서 관리하려는 계획이셨나요?!
제가 이 폴더의 존재가 잘 기억이 안나서요...

Copy link
Collaborator

@bsko98 bsko98 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다 LGTM 👍👍
간단한 질문 남겨뒀습니다!


- name: Build (Gradle) & Docker build/tag
run: |
./gradlew clean build -x test
Copy link
Collaborator

Choose a reason for hiding this comment

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

테스트를 제외하고 빌드한다 라는 내용으로 알고있는데, dev에 머지할 때 테스트 문제는 다 해결되니까 이렇게 하신거라고 생각하면 될까요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

넵! dev 브랜치에는 이미 CI로 테스트가 확인된 상태로 올라가게됩니다.
또한 CD 프로세스 상에 필요한 것은 테스트가 아닌 jar 파일 빌드기 때문에 테스트를 진행하 않는 것이 빠른 프로세스 진행에 도움이 될 것이라고 생각했습니다!

@LeeShinHaeng
Copy link
Contributor Author

사용하지 않는 local-run 폴더의 docker-compose를 삭제했습니다.

혹시 이 폴더에 local용 docker-compose를 이 폴더에서 관리하려는 계획이셨나요?! 제가 이 폴더의 존재가 잘 기억이 안나서요...

#133 에서 추가했던 파일인데, 루트의 docker-local-compose.yaml과 중복된다고 생각하여 이번 PR에서 삭제조치했습니다!

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR sets up continuous deployment for the production server by adding a new GitHub Actions workflow, updating deprecated Spring Boot properties in the application configuration, and removing an unused local-run docker-compose file.

Key Changes:

  • Added production CD workflow that builds Docker images and deploys to production server via SSH
  • Updated Redis configuration from deprecated spring.redis to spring.data.redis structure
  • Removed unused local-run/docker-compose.yml file

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.

File Description
.github/workflows/deploy-with-docker-product.yml New production deployment workflow that builds/pushes Docker images and deploys to prod server using docker-compose
src/main/resources/application.yml Updated Redis config to use spring.data.redis instead of deprecated spring.redis, removed duplicate spring.resources property, and modified prod profile Redis settings to use hardcoded values
local-run/docker-compose.yml Removed entire file as it's no longer needed

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@bsko98 bsko98 left a comment

Choose a reason for hiding this comment

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

코파일럿 코드리뷰 반영에 상현이가 부탁했던 더미데이터 변경까지 너무 고생하셨습니다!

@LeeShinHaeng LeeShinHaeng merged commit ee1eb31 into dev Dec 26, 2025
2 checks passed
@LeeShinHaeng LeeShinHaeng deleted the chore/product-server-cd branch December 26, 2025 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🗂️chore 간단한 업무

Projects

None yet

Development

Successfully merging this pull request may close these issues.

배포서버용 CD 파일 생성 로컬 실행용 도커 컴포즈 파일 생성

3 participants