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

feat: moonyoungCHAE web server #23

Merged
merged 1 commit into from
Aug 16, 2023

Conversation

moonyoungCHAE
Copy link
Contributor

@moonyoungCHAE moonyoungCHAE commented Aug 16, 2023

리뷰 감사합니다 :D

https://thearchivelog.dev/article/optimize-docker-image/
여기에 불필요한 레이어 줄이기 보고 go mod download & go build 같이 실행하긴 했는데,
묶였을 때 캐시 히트가 달라지지 않는 경우에만 묶는다(?)라고 생각하면 되겠죠??

Signed-off-by: moonyoungCHAE <xpf_fl@naver.com>
@GeunSam2
Copy link
Member

네 정확해요. 레이어의 갯수 관점에서만 보면 문영님이 작성하신 방식으로 하는게 줄어들 수 있는데, 결국 레이어 하나의 크기가 커지는 것 뿐, 캐싱 관점에서 보면 오히려 마이너스 요소로 작용해서, 이 경우에는 분리하는게 좋은 판단이에요.

모듈을 다운로드 받는 레이어들이 캐싱되도록 하면, 모듈의 목록이 변경되지 않고 단순 코드만 변경된 경우에는 모듈 다운로드에 필요한 빌드시간을 단축할 수 있어서 이렇게 작성하는게 좋아요.

@GeunSam2 GeunSam2 merged commit 7e73c57 into ArgoWorkflows-OSS:main Aug 16, 2023
2 checks 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.

None yet

2 participants