Skip to content
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

[BE] 컨테이너에 로그 폴더 마운트 볼륨 추가 #175

Merged
merged 1 commit into from
Aug 7, 2024

Conversation

ikjo39
Copy link
Contributor

@ikjo39 ikjo39 commented Aug 7, 2024

관련 이슈

작업 내용

  • backend-cd.yml 중 docker 실행 라인 내 volume 설정 추가했습니다.
  • 로그를 ec2 내부에 써내려야하기에 ro설정을 하지 않았습니다.
  • $HOME이 어떤 걸 뜻하는지 명확히 알지 못하여 ec2의 루트임을 예상하여 다음과 같이 설정하였습니다.

리뷰 요구사항 (선택)

리뷰 중점 사항: 리뷰어가 특히 집중해서 봐야 할 부분이 있나요?

  • docker에 관한 지식이 부족하여 관련 설정이 유효할지 궁금합니다!

@ikjo39 ikjo39 self-assigned this Aug 7, 2024
@ikjo39 ikjo39 added 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :) ⚙️ 환경설정 환경을 설정해요 :) labels Aug 7, 2024
@ikjo39 ikjo39 added this to the 3차 데모데이 milestone Aug 7, 2024
@@ -113,6 +113,7 @@ jobs:
docker run --name momo-api
--rm -d -p 80:8080
--volume=$HOME/security:/momo/security:ro
--volume=$HOME/logs:/momo/logs
Copy link
Contributor

Choose a reason for hiding this comment

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

쓰기 권한임을 명시해야할 것 같아요!

Suggested change
--volume=$HOME/logs:/momo/logs
--volume=$HOME/logs:/momo/logs:rw

Copy link
Contributor

Choose a reason for hiding this comment

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

앗.. 찾아보니 권한을 명시하지 않았을 때는 rw가 default네요..😅

Copy link
Member

Choose a reason for hiding this comment

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

$HOME 은 기본 환경변수로, ~/ 경로를 나타냅니다.
현재 배포된 서버 기준으로 절대경로는 /home/ubuntu 경로가 되겠네요.

설정해 주신대로라면 호스트 머신의 /home/ubuntu/logs 경로가, 컨테이너 내의 /momo/logs 경로로 마운트됩니다.

@@ -113,6 +113,7 @@ jobs:
docker run --name momo-api
--rm -d -p 80:8080
--volume=$HOME/security:/momo/security:ro
--volume=$HOME/logs:/momo/logs
Copy link
Member

Choose a reason for hiding this comment

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

$HOME 은 기본 환경변수로, ~/ 경로를 나타냅니다.
현재 배포된 서버 기준으로 절대경로는 /home/ubuntu 경로가 되겠네요.

설정해 주신대로라면 호스트 머신의 /home/ubuntu/logs 경로가, 컨테이너 내의 /momo/logs 경로로 마운트됩니다.

@ikjo39 ikjo39 changed the title [BE] 로그 폴더 마운트 할 볼륨 컨테이너 추가 [BE] 컨테이어에 로그 폴더 마운트 볼륨 추가 Aug 7, 2024
Copy link
Contributor

@seunghye218 seunghye218 left a comment

Choose a reason for hiding this comment

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

파일 수정하는 김에 포트포워딩 설정도 같이하면 별로이려나요?

@hw0603 hw0603 changed the title [BE] 컨테이어에 로그 폴더 마운트 볼륨 추가 [BE] 컨테이너에 로그 폴더 마운트 볼륨 추가 Aug 7, 2024
@ikjo39 ikjo39 merged commit 1fd179f into develop Aug 7, 2024
4 checks passed
@ikjo39 ikjo39 deleted the chore/173-set-volume branch August 7, 2024 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ 환경설정 환경을 설정해요 :) 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[BE] 도커 내 로그 폴더 관련 볼륨 설정 추가를 위해 CD yaml을 수정해요 :)
5 participants