Skip to content

[chore][web-sample] docker-compose.yaml 추가#33

Merged
eGovFrameSupport merged 2 commits into
eGovFramework:mainfrom
dasomel-eGovFramework:chore/docker-compose-main
Jun 5, 2026
Merged

[chore][web-sample] docker-compose.yaml 추가#33
eGovFrameSupport merged 2 commits into
eGovFramework:mainfrom
dasomel-eGovFramework:chore/docker-compose-main

Conversation

@dasomel
Copy link
Copy Markdown
Contributor

@dasomel dasomel commented May 27, 2026

변경 사유

Dockerfile은 존재하지만 docker-compose.yaml이 없어 로컬 개발 및 단순 배포 시 컨테이너 구동 방법이 불편했다. docker-compose로 한 번에 빌드·구동할 수 있는 환경을 제공한다.

변경 내용

  • docker-compose.yaml 신규 추가
    • Dockerfile 기반 이미지 빌드 및 컨테이너 구동
    • 포트 8080 노출
    • JAVA_OPTS 환경 변수로 메모리 튜닝 지원
    • healthcheck 설정으로 애플리케이션 기동 상태 모니터링

영향 범위

  • 신규 파일 추가만이며 기존 소스 변경 없음

체크리스트

  • 단일 주제만 다룸
  • 기존 동작에 영향 없음
  • 테스트 통과 확인 (빌드 무관 파일)

Dockerfile 기반 컨테이너 빌드·구동 환경을 docker-compose로 제공한다.
healthcheck로 애플리케이션 기동 상태를 주기적으로 확인하며,
JAVA_OPTS를 환경 변수로 노출해 메모리 튜닝을 용이하게 한다.
@eGovFrameSupport
Copy link
Copy Markdown
Contributor

#35(Docker 빌드 복구)가 반영된 main에서 본 PR의 docker-compose.yaml로 로컬 실행을 검증해 정상 동작을 확인하였습니다.

다만 병합 전 아래 보강을 요청드립니다.

  1. clean clone에서의 첫 실행 : 현재 Dockerfile은 COPY ./target/egovframe-web-5.0.0.war
    를 하는 단일 스테이지라, target/에 WAR가 없는 상태(막 clone한 상태)에서는
    docker compose up --build가 COPY 단계에서 실패합니다. 즉 사전에 mvn package
    필요합니다. compose의 취지(한 번에 기동)에 맞게 mvn package 선행을 README/주석에
    명시하거나, 빌드를 멀티스테이지로 바꿔 compose만으로 완결되게 정리 부탁드립니다.

  2. 이미지 이름 일관성 : 본 compose는 egovframe-web-sample:5.0.0을 쓰는데, k8s
    매니페스트/문서는 egovframe-web:5.0.0을 사용합니다. 같은 산출물의 이름이 둘로
    갈리므로 하나로 통일 부탁드립니다.

  3. version: "3.8" 은 현재 Compose에서 폐기되어 경고가 출력됩니다.

위 사항 반영되면 병합을 진행하도록 하겠습니다. 감사합니다.

- compose 이미지명을 k8s 매니페스트/문서와 동일하게 egovframe-web:5.0.0 으로 통일
- Compose v2에서 폐기되어 경고를 유발하는 version 키 제거
- 클린 클론에서 docker compose up --build 시 사전 mvn package 가 필요함을 주석으로 명시
@dasomel
Copy link
Copy Markdown
Contributor Author

dasomel commented Jun 2, 2026

리뷰 감사합니다. 요청하신 보강 사항을 다음과 같이 반영했습니다.

  1. 클린 클론 첫 실행: Dockerfile이 빌드 산출물 WAR를 복사하는 구성이라 docker compose up --build 이전에 mvn package로 WAR를 먼저 생성해야 함을 docker-compose.yaml 상단 주석으로 명시했습니다. 멀티스테이지 전환은 빌드 구성 변경이 함께 필요해 본 PR(compose 추가) 범위를 벗어나므로 별도로 분리하는 편이 적절하다고 판단했습니다.
  2. 이미지 이름: compose의 image를 k8s 매니페스트/문서와 동일하게 egovframe-web:5.0.0으로 통일했습니다.
  3. version 키: Compose v2에서 폐기되어 경고를 유발하는 version: "3.8"을 제거했습니다.

확인 부탁드립니다. 감사합니다.

@eGovFrameSupport
Copy link
Copy Markdown
Contributor

표준프레임워크에 대한 지속적인 참여에
대단히 감사드립니다.

@eGovFrameSupport eGovFrameSupport merged commit fce32a7 into eGovFramework:main Jun 5, 2026
1 check 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.

2 participants