Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

2-5 [Fix] Frontend Dockerfile 수정 #112

Merged
merged 2 commits into from
Dec 9, 2022
Merged

2-5 [Fix] Frontend Dockerfile 수정 #112

merged 2 commits into from
Dec 9, 2022

Conversation

leesungbin
Copy link
Member

@leesungbin leesungbin commented Dec 9, 2022

개요

작업사항

  • Dockerfile 내에 robots.txt 삭제

리뷰 요청사항

  • 빌드가 실패하는 문제가 자주 있지는 않지만, 한번 실패하면 다시 PR을 날려야되는 문제가 있네요.
  • 로컬에서 빌드 테스트를 한번 해보고 PR을 올려도 되긴 하지만, 빌드했던 이미지들이 차지하는 용량으로인해 삭제도 해줘야하고 귀찮은 부분이 조금 있습니다.
  • PR을 날린 시점에 이미지를 container registry에 push 하고, merge 될 때 배포되는 서버에 접속해서 docker compose pull 하는 방식을 사용할 순 있을 것 같습니다.
    • 이렇게했을 때 장점
      • PR을 머지하는 시점에서 항상 성공한 빌드 이미지만 사용할 수 있어요.
      • PR 올라갈 때 이미지 빌드 CI가 돌아서, merge 하는 시점에는 배포되는 서버가 이미지를 pull만 하면 되므로 배포에 소요되는 시간이 체감상 확 줄어듭니다.
    • 예상되는 문제점
      • Container registry에 이미지가 꽤 많이 쌓일 것 같습니다. (PR 날린 이후의 커밋들에 대해서도 빌드해야하니까)
      • docker compose pull 할 때 어떤 이미지를 가져와야하는지(어떤 tag인지) 알아와야하는 로직이 필요한데 바로 떠오르진 않네요.
  • 다른 팀들은 어떻게 하고 있나 조사를 한번 해보겠습니다~

@github-actions github-actions bot requested review from JunYupK and Palwol December 9, 2022 00:42
@yeynii
Copy link
Member

yeynii commented Dec 9, 2022

이전 PR의 action을 다시 수행하고 싶으면 action 탭 들어가서 해당 action누르고 re run all-jobs 누르면 되긴 합니다.

image

https://github.com/boostcampwm-2022/web18-PRV/actions/runs/3653374455

@leesungbin
Copy link
Member Author

이전 PR의 action을 다시 수행하고 싶으면 action 탭 들어가서 해당 action누르고 re run all-jobs 누르면 되긴 합니다.

image

https://github.com/boostcampwm-2022/web18-PRV/actions/runs/3653374455

지금 같은 경우에는 dev에 머지된 코드로 deploy action이 돌아가다보니까, 과거에 머지된 PR을 다시 Run 해도 똑같긴 합니다 ㅜ

@Palwol
Copy link
Collaborator

Palwol commented Dec 9, 2022

이 PR이 먼저 머지가 됐어야했는데, 지금은 FE 빌드가 실패하는군요ㅜ
혹시 이 PR을 머지하고 나서 빌드 실패한 PR은 다시 PR 올려서 머지하는것밖에 방법이 없을까요?

Copy link
Collaborator

@JunYupK JunYupK left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@leesungbin leesungbin merged commit 206df2c into dev Dec 9, 2022
@leesungbin leesungbin deleted the ci/docker-compose branch December 9, 2022 07:08
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants