Skip to content

Conversation

@minkyu97
Copy link
Collaborator

@minkyu97 minkyu97 commented Jul 31, 2024

추가된 기능

ci 액션에 경로 필터를 추가하고 도커 빌드 캐시를 활용하여 시간을 단축시킵니다.

테스트 방법

관련 없는 경로의 파일만 수정한 뒤 PR 올려서 ci 액션 안 돌아가는지 보기
ci 액션 잘 돌아가는지 확인

관련 링크

https://docs.github.com/en/actions/writing-workflows/workflow-syntax-for-github-actions#onpushpull_requestpull_request_targetpathspaths-ignore
https://docs.docker.com/build/ci/github-actions/cache/#github-cache

@minkyu97 minkyu97 self-assigned this Jul 31, 2024
@minkyu97 minkyu97 requested review from jj1kim and woohm402 as code owners July 31, 2024 15:40
@minkyu97
Copy link
Collaborator Author

ci.yml 수정했는데 안 돌아가는 거 확인

@minkyu97
Copy link
Collaborator Author

Cargo.toml 수정한 f6ff5b5 에서 ci 액션 돌아가는 거 확인

Copy link
Member

@woohm402 woohm402 left a comment

Choose a reason for hiding this comment

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

(사실 저는 개인적으로 *.md 빼고는 뭐가 수정돼도 CI 돌아야 된다고 생각하긴 합니다)

@minkyu97 minkyu97 changed the title feat: ci 액션에 경로 필터 추가 feat: ci 액션에 경로 필터 추가 & 도커 빌드 캐시 추가 Jul 31, 2024
Comment on lines +25 to +35
- name: Set up Docker Buildx
uses: docker/setup-buildx-action@v3
- name: Build
uses: docker/build-push-action@v6
with:
context: .
push: false
tags: waffle-judge
cache-from: type=gha
cache-to: type=gha,mode=max
outputs: type=docker,dest=/tmp/waffle-judge.tar
Copy link
Collaborator Author

@minkyu97 minkyu97 Jul 31, 2024

Choose a reason for hiding this comment

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

별도 PR로 올릴까 하다가.. ci 개선이라는 명목으로 퉁쳐서 도커 공식문서에 있는 깃헙 액션 프리셋을 활용하도록 변경했습니다.
당장은 캐시가 없고 앞뒤 준비과정이 있어서 조금 더 느려진 감이 있는데 캐시가 활용되면 더 빨라질 거예요

@minkyu97 minkyu97 requested a review from woohm402 July 31, 2024 18:41
@minkyu97
Copy link
Collaborator Author

https://github.com/wafflestudio/waffle-judge/actions/runs/10185735551/attempts/1#summary-28175933541
캐시 기능 말고도 좋은 점이, 요런 식으로 빌드 결과 요약도 첨부해줍니다

@woohm402
Copy link
Member

woohm402 commented Aug 1, 2024

옹 그럼 지금은 캐시가 올라가 있을 테니 테스트커밋 하나 추가해서 속도 얼마나 개선됐는지 한번 볼 수 있을까요?

@minkyu97
Copy link
Collaborator Author

minkyu97 commented Aug 1, 2024

뭐야 왜 캐시 작동 안해

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.

3 participants