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

AWS ECR Public 이미지를 사용하도록 Dockerfile 수정 #65

Merged

Conversation

yuminn-k
Copy link
Contributor

🔍 이 PR로 해결하고자 하는 문제는 무엇인가요?

  • Docker Hub의 pull rate limit 문제를 해결하기 위해 AWS ECR Public 이미지를 사용하도록 Dockerfile을 수정했습니다.

✨ 이 PR로 주요하게 바뀐 점은 무엇인가요?

  • Dockerfile의 base 이미지를 AWS ECR Public 이미지로 변경했습니다.
  • GitHub Actions 워크플로우에서 AWS ECR에 로그인하도록 수정했습니다.

🔖 주요 변경 사항 외에 추가로 변경된 부분이 있나요?

  • 없음

🙏🏻 리뷰어가 특히 봐주었으면 하는 부분은 무엇인가요?

  • Dockerfile과 GitHub Actions 워크플로우의 변경 사항이 올바르게 적용되었는지 확인 부탁드립니다.

🩺 이 PR로 테스트나 검증이 필요한 부분이 있나요?

  • AWS ECR Public 이미지를 사용하여 Docker 이미지를 빌드하고 푸시하는 과정이 정상적으로 작동하는지 확인이 필요합니다.

📚 관련된 Issue나 Notion, 문서

🖥 작동하는 모습

스크린샷이나 녹화된 비디오, 또는 gif를 추가해서, 리뷰어가 변경점을 이해하는 데 도움이 되도록 해주세요.

📌 PR을 작성할 때의 주의사항

  • 리뷰어는 코드 리뷰 시 좋은 코드의 방향성을 제시하지만, 코드 수정을 강요하지 않습니다.
  • 리뷰어는 좋은 코드를 발견하면 칭찬과 격려를 아끼지 않습니다.
  • 리뷰는 특별한 경우가 아니라면, 리뷰어로 지정된 시점부터 3일 이내에 완료해주세요.
  • 댓글 작성 시 Prefix에 P1, P2, P3을 작성해주시면, 담당자가 댓글에 대해 더 명확히 대응할 수 있습니다.
    • P1 : 반드시 반영해주세요 (Request Changes) - 문제가 발생하거나, 취약점이 발견된 경우 등.
    • P2 : 반영을 적극적으로 검토해주세요 (댓글).
    • P3 : 이런 방법도 있을 것 같다는 사소한 의견입니다 (Chore).

- Changed base images to use AWS ECR Public to avoid Docker Hub rate limits
- Updated build and execution stages accordingly

Related to ryokushaka#37
@yuminn-k yuminn-k added ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🐞 BugFix Something isn't working 🆗 safe CI workflow 동작 관련 🌏 Deploy 배포 관련 labels Oct 14, 2024
@yuminn-k yuminn-k self-assigned this Oct 14, 2024
@yuminn-k yuminn-k merged commit 5c1a1fb into ryokushaka:main Oct 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix Something isn't working 🌏 Deploy 배포 관련 ⏰ P0 반드시 해결해야 하는 중요한 이슈나 문제 (예: 버그, 취약점 등) 🆗 safe CI workflow 동작 관련
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant