-
Notifications
You must be signed in to change notification settings - Fork 1
feat: 헬스체크용 이미지 업로드 #133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: 헬스체크용 이미지 업로드 #133
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
bsko98
left a comment
There was a problem hiding this 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 설정에 볼륨 설정을 추천하던데 혹시 설정 안 해두신 이유가 따로 있으신지 궁금합니다.
볼륨 설정시 컴퓨터 내부에 특정 위치에 마운트를 걸어야하는데, OS 별로 파일 시스템의 표현방법이 다릅니다. 또한, 저희는 현재 flyway를 사용하고 있기 때문에 새 컨테이너를 올릴 경우엔 알아서 초기화 되기 때문에 영속성 관리를 위해서 볼륨을 사용하지 않아도 된다고 생각했습니다! 😄 |
docker-compose 파일은 개인 로컬용 컴포즈 파일이고, 개인의 컴퓨터별 OS의 파일 시스템 표현이 다르기 때문에 여기서 설정할 수 없다 라고 이해했는데 혹시 맞을까요??
이 부분에 대해서는 저도 동의합니다! 저희가 사용하는 테스트 데이터는 사실 R__ 파일에 다 있기 때문에 볼륨의 역할이 조금 약해졌다고 생각합니다! |
네! 윈도우는 경로가 "C:\Users\ ..." 이렇게 표현하지만 리눅스 계열은 "/Users/..." 이렇게 표현하기 때문에 모든 OS에 적용하기 위해서는 절대 경로로 사용할 수 없습니다! |
bsko98
left a comment
There was a problem hiding this 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 입니다 👍👍
📋 상세 설명
📊 체크리스트
📆 마감일