Skip to content

chore: Github Actions CI/CD 파이프라인 구축#53

Merged
Woomin-Wang merged 5 commits into
mainfrom
chore/ci-cd
Apr 21, 2026
Merged

chore: Github Actions CI/CD 파이프라인 구축#53
Woomin-Wang merged 5 commits into
mainfrom
chore/ci-cd

Conversation

@Woomin-Wang
Copy link
Copy Markdown
Contributor

📝 작업 내용 (Description)

  • Github Actions CI/CD 파이프라인 구축

🔄 변경 유형 (Type of Change)

  • ✨ 새로운 기능 (feat)
  • 🐛 버그 수정 (fix)
  • 📝 문서 수정 (docs)
  • 💄 스타일 (style)
  • ♻️ 리팩토링 (refactor)
  • ✅ 테스트 (test)
  • 🔧 기타 (chore)

✅ 체크리스트 (Checklist)

  • 코드가 정상적으로 동작하는지 확인했습니다
  • 기존 테스트가 통과합니다
  • 필요한 경우 새로운 테스트를 추가했습니다

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces production-ready Docker configurations, including a new docker-compose-prod.yml with Fluent-Bit logging and PostgreSQL, and adds Spring Boot Actuator for health monitoring. Feedback was provided regarding the Docker health check interval, which was set to an impractical 24 hours, and the use of the Actuator health endpoint. Additionally, corrections were suggested for incorrect volume paths in the production compose file and a suspicious reference to Dockerfile.prev in the development configuration.

Comment thread Dockerfile
Comment on lines +36 to +37
HEALTHCHECK --interval=24h --timeout=10s --start-period=40s --retries=3 \
CMD curl -sf http://localhost:7300/health || exit 1
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

HEALTHCHECK의 interval이 24시간(24h)으로 설정되어 있습니다. 이는 컨테이너의 상태 변화를 감지하기에 너무 긴 시간입니다. 일반적으로 30초(30s) 또는 1분(1m) 정도로 설정하는 것이 적절합니다. 또한, Spring Boot Actuator를 의존성에 추가하셨으므로 기본 헬스체크 경로인 /actuator/health를 사용하는 것이 좋습니다.

HEALTHCHECK --interval=30s --timeout=10s --start-period=40s --retries=3 \
  CMD curl -sf http://localhost:7300/actuator/health || exit 1

Comment thread docker-compose-prod.yml
Comment on lines +6 to +7
- ./fluentbit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro
- ./fluentbit/parsers.conf:/fluent-bit/etc/parsers.conf:ro
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

high

Fluent-Bit 설정 파일의 호스트 경로가 프로젝트 구조와 일치하지 않습니다. 현재 프로젝트 구조상 설정 파일은 ./docker/fluent-bit/ 디렉토리에 위치하므로 이를 반영해야 합니다.

      - ./docker/fluent-bit/fluent-bit.conf:/fluent-bit/etc/fluent-bit.conf:ro
      - ./docker/fluent-bit/parsers.conf:/fluent-bit/etc/parsers.conf:ro

Comment thread docker-compose-dev.yml
build:
context: .
dockerfile: Dockerfile
dockerfile: Dockerfile.prev
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

Dockerfile.prev 파일이 프로젝트에 존재하지 않는 것으로 보입니다. 기존의 Dockerfile을 사용하도록 수정하거나, 해당 파일이 의도적으로 생성된 것인지 확인이 필요합니다.

      dockerfile: Dockerfile

@Woomin-Wang Woomin-Wang merged commit 770ac61 into main Apr 21, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant