-
Notifications
You must be signed in to change notification settings - Fork 30
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: D0ri123 helm container #39
Conversation
Signed-off-by: LeeSeolHui <lsh81550@gmail.com>
Signed-off-by: LeeSeolHui <lsh81550@gmail.com>
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.
현재 Dockerfile 구조로는 빌드된 이미지가 상대적으로 클 수 있어요.
Multi Stage Build에 대해 학습하고 적용하면 merge 진행하도록 하겠습니다.
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.
앗 도커파일에서 이미지 용량도 고려해야 하는 부분이었군욥..! 감사합니다 :)
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.
추가로 도커파일을 구성하면서 문득 든 궁금한 점입니다만 혹시 COPY 명령어와 RUN cp의 차이가 뭔지 여쭤봐도 될까요??
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.
RUN cp론 외부 filesystem에 접근 못하지 않나요??
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.
copy는 호스트 머신과의 컨텍스트 공유면
run cp는 빌드되고 있는 컨테이너 내부 파일시스템이거든요
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.
아핫 그러니깐 제 컴퓨터에 있는 파일을 복사한다고 할 때는 COPY를 써야 하는 거고,
컨테이너 내부에서 파일을 복사한다고 할 때에는 RUN cp를 사용해야 한다는게 맞는걸까요??
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.
네 그렇게 사용하면 될것 같아요
Signed-off-by: LeeSeolHui <lsh81550@gmail.com>
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.
LGTM
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.
RUN cp론 외부 filesystem에 접근 못하지 않나요??
close #34