Skip to content

Conversation

@dev-ant
Copy link
Contributor

@dev-ant dev-ant commented Jul 24, 2025

🛠️ 인프라 변화

  • 클라우드 서비스에 지속적 배포를 위한 CD 파이프 라인을 구축한다

📝 상세 설명

  • Docker Compose + Github Actions를 사용
  • 개발 중에 운영 환경과 같은 횟수의 OPEN API 호출은 큰 의미가 없음
    • 경매장 거래 내역 호출 시점을 매시간 0분 0초 -> 4시간마다 0분 0초로 변경

🧐 예상 결과

  • NextJs 서버와 실제 API로 통신 가능

📊 체크리스트

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

📆 마감일

Close #39

@dev-ant dev-ant requested a review from Copilot July 24, 2025 15:32
@dev-ant dev-ant self-assigned this Jul 24, 2025
@dev-ant dev-ant added the 🏗️infrastructure 인프라 구조 설정 label Jul 24, 2025

This comment was marked as outdated.

@github-actions
Copy link

github-actions bot commented Jul 24, 2025

✅ 테스트 결과 for PR

Build: success

🧪 테스트 실행 with Gradle
📈 Coverage: -0.00%

📁 테스트 결과
📁 커버리지 보고서 (HTML)

@codecov
Copy link

codecov bot commented Jul 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@dev-ant dev-ant closed this Jul 24, 2025
@dev-ant dev-ant reopened this Jul 24, 2025
@dev-ant dev-ant requested a review from Copilot July 24, 2025 16:39

This comment was marked as outdated.

@dev-ant dev-ant closed this Jul 30, 2025
@dev-ant dev-ant reopened this Jul 30, 2025
@dev-ant dev-ant closed this Jul 30, 2025
@dev-ant dev-ant reopened this Jul 30, 2025
@dev-ant dev-ant requested a review from Copilot July 30, 2025 03:03

This comment was marked as outdated.

@dev-ant dev-ant requested a review from Copilot July 31, 2025 13:39
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 establishes a CD pipeline using GitHub Actions and Docker Compose for continuous deployment to cloud services. It includes infrastructure configuration changes to support production-like environments while optimizing API call frequency for development.

Key Changes:

  • Added GitHub Actions workflow for automated Docker build and deployment
  • Created Docker Compose configuration for containerized deployment
  • Modified auction history API call frequency from hourly to every 4 hours
  • Removed default values from JWT configuration properties

Reviewed Changes

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

Show a summary per file
File Description
.github/workflows/push-cd-dev.yml GitHub Actions CD pipeline with Docker build, push, and remote deployment
docker-compose-dev.yaml Docker Compose configuration for Spring application with health checks and logging
Dockerfile Simple Docker image configuration using OpenJDK 21
.dockerignore Docker build context exclusions for security and optimization
src/main/resources/application-sample.yml Updated cron schedule and removed JWT default values

auction-history:
delay-ms: 1000
cron: "0 0 */1 * * *" # 1시간마다 실행
cron: "0 0 */4 * * *" # 1시간마다 실행
Copy link

Copilot AI Jul 31, 2025

Choose a reason for hiding this comment

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

The comment says '1시간마다 실행' (runs every 1 hour) but the cron expression '0 0 */4 * * *' runs every 4 hours. The comment should be updated to '4시간마다 실행' to match the actual schedule.

Suggested change
cron: "0 0 */4 * * *" # 1시간마다 실행
cron: "0 0 */4 * * *" # 4시간마다 실행

Copilot uses AI. Check for mistakes.
- name: Deploy and Restart Container
run: |
ssh -i ~/.ssh/my-key.pem ${{ secrets.SERVER_USER }}@${{ secrets.SERVER_HOST }} << 'EOF'
mkdir -p /home/${{ secrets.SERVER_USER }}/app
Copy link

Copilot AI Jul 31, 2025

Choose a reason for hiding this comment

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

The directory creation is duplicated - it's already done on line 51. This redundant command should be removed to avoid unnecessary operations.

Suggested change
mkdir -p /home/${{ secrets.SERVER_USER }}/app

Copilot uses AI. Check for mistakes.
Comment on lines +32 to +33
.gradle/

Copy link

Copilot AI Jul 31, 2025

Choose a reason for hiding this comment

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

The '.gradle/' directory is listed twice (lines 30 and 32). Remove the duplicate entry to clean up the file.

Suggested change
.gradle/

Copilot uses AI. Check for mistakes.
@dev-ant dev-ant merged commit 6c65e91 into dev Jul 31, 2025
2 checks passed
@dev-ant dev-ant deleted the infra/set-cd-pipeline branch July 31, 2025 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏗️infrastructure 인프라 구조 설정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Github Actions Docker Compose CD 파이프 라인 구축

2 participants