Skip to content

chore: Docker 설정 간소화 및 CI/CD 안정화#67

Merged
Woomin-Wang merged 2 commits into
mainfrom
chore/simplify-docker-config
May 4, 2026
Merged

chore: Docker 설정 간소화 및 CI/CD 안정화#67
Woomin-Wang merged 2 commits into
mainfrom
chore/simplify-docker-config

Conversation

@Woomin-Wang
Copy link
Copy Markdown
Contributor

📌 관련 이슈 (Related Issue)

📝 작업 내용 (Description)

Docker 설정 간소화

  • docker-compose-dev.ymldocker-compose.yaml로 이름 변경 및 간소화
  • docker/ 하위 프로덕션 설정을 deploy/로 이동하여 배포 관련 파일 경로 통일
  • Dockerfile 불필요한 레이어 제거
  • .dockerignore 추가로 이미지 빌드 컨텍스트 최적화

CI/CD 안정화

파일 확장자 통일

  • application-*.yml.yaml, docker-compose.yml.yaml로 통일

🔄 변경 유형 (Type of Change)

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

✅ 체크리스트 (Checklist)

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

💬 추가 코멘트 (Additional Comments)

  • docker image prune--filter "reference="는 dangling 이미지에만 적용되어 태그가 붙은 이전 이미지는 삭제되지 않았습니다.
    수정 후에는 docker images로 대상 이미지를 명시적으로 조회하여 최신 1개만 남기고 나머지를 삭제합니다.
  • QEMU는 GitHub 호스팅 런너에 기본 설치되어 있어 기존에도 동작했으나, 런너 이미지 업데이트 시 깨질 수 있어 명시적으로 추가했습니다.

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 refactors the Docker configuration and updates the application's naming conventions to 'prepair-backend-java-api'. Key changes include an expanded .dockerignore, a more frequent Docker health check, and the addition of a PostgreSQL health check in the docker-compose file to ensure proper service orchestration. A review comment suggests improving the database health check by specifying the database name in the pg_isready command for better reliability.

Comment thread docker-compose.yaml
volumes:
- postgres-data:/var/lib/postgresql/data
healthcheck:
test: ["CMD-SHELL", "pg_isready -U ${POSTGRES_USER}"]
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

PostgreSQL 상태 확인 시, 서버가 연결을 수락하는지 확인하는 것뿐만 아니라, 애플리케이션이 사용할 특정 데이터베이스가 준비되었는지 확인하는 것이 더 안정적입니다. pg_isready-d 플래그를 추가하여 데이터베이스 이름을 명시하는 것을 권장합니다.

      test: ["CMD-SHELL", "pg_isready -d ${POSTGRES_DB} -U ${POSTGRES_USER}"]

@Woomin-Wang Woomin-Wang merged commit 47c194f into main May 4, 2026
4 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.

fix: docker image prune의 reference 필터 미지원으로 인한 의도치 않은 전체 이미지 삭제

1 participant