Skip to content

chore: Docker Compose 기반 배포 파이프라인 구성#34

Merged
Sean-mn merged 4 commits intodevelopfrom
cicd/cd-pipeline
Apr 3, 2026
Merged

chore: Docker Compose 기반 배포 파이프라인 구성#34
Sean-mn merged 4 commits intodevelopfrom
cicd/cd-pipeline

Conversation

@Sean-mn
Copy link
Copy Markdown
Contributor

@Sean-mn Sean-mn commented Apr 3, 2026

📚작업 내용

  • 배포 방식을 docker run 단일 컨테이너에서 Docker Compose로 전환
  • GitHub Actions 워크플로우에 SCP로 compose 파일 전송 스텝 추가
  • SSH 배포 스크립트를 docker compose pull/up 방식으로 교체
  • compose.prod.yaml에서 DB 컨테이너 분리 (외부 DB 사용)
  • compose.prod.yaml에 Redis 컨테이너 및 내부 네트워크 추가
  • appsettings.json / appsettings.Production.json의 민감 정보 플레이스홀더 제거 (공백으로 교체)
  • 환경변수명 오타 수정: STEAM_API_KEY → STEAM_WEB_API_KEY

◀️참고 사항

DB는 compose에서 분리하여 외부 PostgreSQL 인스턴스를 사용합니다. Redis는 compose 내부 서비스로 운영하며, pushandpull-network 브릿지 네트워크로 연결됩니다. 시크릿(DB_CONNECTION_STRING, REDIS_CONNECTION_STRING, STEAM_WEB_API_KEY)은 GitHub Secrets에서 SSH 배포 시 .env 파일로 주입됩니다.

✅체크리스트

[ ]안에 x를 작성하면 체크박스를 체크할 수 있습니다.

  • 현재 의도하고자 하는 기능이 정상적으로 작동하나요?
  • 변경한 기능이 다른 기능을 깨뜨리지 않나요?

추후 필요한 체크리스트는 업데이트 될 예정입니다.

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 production deployment configuration by removing the local PostgreSQL database service, renaming existing services for consistency, and introducing a dedicated Docker network. It also clears hardcoded connection strings and API keys from the application settings. Feedback includes a critical observation regarding a missing environment variable for the external database connection string in the Docker Compose file, which would cause a runtime exception, and a suggestion to align the server's container name with its new service name.

@Sean-mn Sean-mn merged commit 1101985 into develop Apr 3, 2026
4 checks passed
@Sean-mn Sean-mn deleted the cicd/cd-pipeline branch April 3, 2026 07:04
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