Skip to content

fix: docker-compose.prod.yml 경로 및 서버 설정 동기화#56

Merged
Woomin-Wang merged 11 commits into
mainfrom
chore/ci-cd
Apr 22, 2026
Merged

fix: docker-compose.prod.yml 경로 및 서버 설정 동기화#56
Woomin-Wang merged 11 commits into
mainfrom
chore/ci-cd

Conversation

@Woomin-Wang
Copy link
Copy Markdown
Contributor

📝 작업 내용 (Description)

  • docker-compose.prod.yml 경로 및 서버 설정 동기화

🔄 변경 유형 (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 updates the docker-compose.prod.yml configuration by modifying volume paths for the fluent-bit service, removing the public port mapping for the postgres service, and updating the environment file path. A review comment suggests verifying the fluent-bit volume paths, specifically highlighting a potential typo in the directory name and questioning the relative path logic compared to the previous configuration.

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 설정 파일의 경로가 ../fluentbit/로 변경되었습니다. 다음 사항들을 확인해 주세요.

  1. 디렉토리 이름 오타 가능성: 기존에는 fluent-bit(하이픈 포함)이었으나 현재 fluentbit(하이픈 없음)으로 변경되었습니다. 실제 디렉토리 이름이 변경된 것이 아니라면 경로 오류가 발생합니다.
  2. 상대 경로 확인: docker-compose.prod.yml 파일이 docker/ 디렉토리 내부에 있으므로, 설정 파일이 docker/fluent-bit/에 위치한다면 ../fluentbit/가 아닌 ./fluent-bit/가 올바른 경로입니다.

만약 설정 파일들을 프로젝트 루트의 fluentbit/ 디렉토리로 이동시킨 것이라면 현재 설정이 맞지만, 그렇지 않다면 수정이 필요합니다.

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

@Woomin-Wang Woomin-Wang merged commit a0336c6 into main Apr 22, 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.

1 participant