Skip to content

Conversation

@LeeShinHaeng
Copy link
Contributor

📋 상세 설명

  • 헬스체크 및 재시동 이미지를 함께 도커 컴포즈에 업로드했습니다.
  • 헬스체크 API를 위해 Actuator 의존성 추가했습니다.
    • Actuator에 스프링 앱만 감지하도록 설정
  • 로컬에서의 autoheal 확인 및 편리한 실행을 위해 로컬용 docker-compose 파일 업로드했습니다.
    • IntelliJ 실행 대신 이미지를 만들어 로컬에서 local-run/docker-compose.yml 실행으로 테스트도 가능합니다.

📊 체크리스트

  • PR 제목이 형식에 맞나요 e.g. feat: PR을 등록한다
  • 코드가 테스트 되었나요
  • 문서는 업데이트 되었나요
  • 불필요한 코드를 제거했나요
  • 이슈와 라벨이 등록되었나요

📆 마감일

Close #131

@LeeShinHaeng LeeShinHaeng self-assigned this Nov 29, 2025
@LeeShinHaeng LeeShinHaeng added the ✨feature 새로운 기능 추가 label Nov 29, 2025
@LeeShinHaeng LeeShinHaeng linked an issue Nov 29, 2025 that may be closed by this pull request
3 tasks
@github-actions
Copy link

github-actions bot commented Nov 29, 2025

Unit Test Results

  36 files  ±0    36 suites  ±0   4s ⏱️ ±0s
109 tests ±0  109 ✔️ ±0  0 💤 ±0  0 ±0 
110 runs  ±0  110 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit 94c0a1c. ± Comparison against base commit 5ea4fc6.

♻️ This comment has been updated with latest results.

@codecov
Copy link

codecov bot commented Nov 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@LeeShinHaeng LeeShinHaeng marked this pull request as ready for review November 29, 2025 01:54
Copy link
Collaborator

@bsko98 bsko98 left a comment

Choose a reason for hiding this comment

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

고생많으셨습니다!!!
조금 알아봤는데 local-run/docker-compose.yml에
redis와 mysql 설정에 볼륨 설정을 추천하던데 혹시 설정 안 해두신 이유가 따로 있으신지 궁금합니다.

@LeeShinHaeng
Copy link
Contributor Author

고생많으셨습니다!!! 조금 알아봤는데 local-run/docker-compose.yml에 redis와 mysql 설정에 볼륨 설정을 추천하던데 혹시 설정 안 해두신 이유가 따로 있으신지 궁금합니다.

볼륨 설정시 컴퓨터 내부에 특정 위치에 마운트를 걸어야하는데, OS 별로 파일 시스템의 표현방법이 다릅니다.
(윈도우는 , 리눅스 계열은 /)
즉 어떤 OS의 컴퓨터에서는 docker-compose 파일이 정상 동작하지 않을 수 있습니다.

또한, 저희는 현재 flyway를 사용하고 있기 때문에 새 컨테이너를 올릴 경우엔 알아서 초기화 되기 때문에 영속성 관리를 위해서 볼륨을 사용하지 않아도 된다고 생각했습니다! 😄

@bsko98
Copy link
Collaborator

bsko98 commented Dec 1, 2025

볼륨 설정시 컴퓨터 내부에 특정 위치에 마운트를 걸어야하는데, OS 별로 파일 시스템의 표현방법이 다릅니다.
(윈도우는 , 리눅스 계열은 /)
즉 어떤 OS의 컴퓨터에서는 docker-compose 파일이 정상 동작하지 않을 수 있습니다.

docker-compose 파일은 개인 로컬용 컴포즈 파일이고, 개인의 컴퓨터별 OS의 파일 시스템 표현이 다르기 때문에 여기서 설정할 수 없다 라고 이해했는데 혹시 맞을까요??

또한, 저희는 현재 flyway를 사용하고 있기 때문에 새 컨테이너를 올릴 경우엔 알아서 초기화 되기 때문에 영속성 관리를 위해서 볼륨을 사용하지 않아도 된다고 생각했습니다! 😄

이 부분에 대해서는 저도 동의합니다! 저희가 사용하는 테스트 데이터는 사실 R__ 파일에 다 있기 때문에 볼륨의 역할이 조금 약해졌다고 생각합니다!

@LeeShinHaeng
Copy link
Contributor Author

LeeShinHaeng commented Dec 1, 2025

docker-compose 파일은 개인 로컬용 컴포즈 파일이고, 개인의 컴퓨터별 OS의 파일 시스템 표현이 다르기 때문에 여기서 설정할 수 없다 라고 이해했는데 혹시 맞을까요??

네! 윈도우는 경로가 "C:\Users\ ..." 이렇게 표현하지만 리눅스 계열은 "/Users/..." 이렇게 표현하기 때문에 모든 OS에 적용하기 위해서는 절대 경로로 사용할 수 없습니다!
찾아보니 위와 같은 이유로 인해 named volume 이라는 방식이나 절대경로를 사용하는 방식을 사용한다고 합니다.
하지만 공감해주신 것처럼 현재 저희는 flyway를 사용하고 있기 때문에 Volume을 꼭 사용해야할 이유가 부족하다고 생각해 따로 적용하지는 않았습니다!

Copy link
Collaborator

@bsko98 bsko98 left a comment

Choose a reason for hiding this comment

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

네! 윈도우는 경로가 "C:\Users\ ..." 이렇게 표현하지만 리눅스 계열은 "/Users/..." 이렇게 표현하기 때문에 모든 OS에 적용하기 위해서는 절대 경로로 사용할 수 없습니다!
찾아보니 위와 같은 이유로 인해 named volume 이라는 방식이나 절대경로를 사용하는 방식을 사용한다고 합니다.
하지만 공감해주신 것처럼 현재 저희는 flyway를 사용하고 있기 때문에 Volume을 꼭 사용해야할 이유가 부족하다고 생각해 따로 적용하지는 않았습니다!

그렇다면 충분히 이해되는 상황이네요! LGTM 입니다 👍👍

@LeeShinHaeng LeeShinHaeng merged commit cdb3f79 into dev Dec 1, 2025
2 checks passed
@LeeShinHaeng LeeShinHaeng deleted the feat/health-check-and-rerun branch December 1, 2025 13:58
@LeeShinHaeng LeeShinHaeng mentioned this pull request Dec 25, 2025
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨feature 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

unhealthy한 서버 앱을 제어하는 기능 추가

3 participants