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

Add Docker image build and push workflow #2

Open
seokho-son opened this issue Aug 7, 2024 · 9 comments
Open

Add Docker image build and push workflow #2

seokho-son opened this issue Aug 7, 2024 · 9 comments
Assignees

Comments

@seokho-son
Copy link
Member

seokho-son commented Aug 7, 2024

저장소 활용을 위해서, Docker 이미지 빌드 및 푸시 워크플로우가 필요합니다.

하기 예시를 참고하여,
가급적 최소 수정으로 본 저장소의 개발 내용이 컨테이너 이미지로 등록될 수 있도록 깃허브 워크플로우 등록을 요청 드립니다.

@sykim-etri
Copy link

혹시 해당 이슈의 업데이트 일정이 어떻게 될까요?

@pbcccbeatboard-strato
Copy link
Contributor

image
image

현재 테스트중에 있습니다.
해당 액션에서 docker hub, git container registry로그인은 정상적으로 진행되는것으로 보이나
failed to push tempcicd: push access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed 상태로 아직 저 부분을 확인하지 못한 상태입니다.

개인 아이디로 타 프로젝트에서 실행해본결과 별 문제가 없어 조금 더 테스트해본 후 다시 말씀드리도록 하겠습니다.

@sykim-etri
Copy link

@seokho-son @yunkon-kim
혹시 상기 에러에 대해서 코멘트 주실 수 있는 부분이 있으실까요?

@yunkon-kim
Copy link
Member

@pbcccbeatboard-strato @sykim-etri

upstream main에 직접 적용 및 테스트를 추천 드립니다.

(참고)

  • GitHub Actions 워크플로우 구동에 필요한 secrects을 M-CMP organization 설정에 적용해 둔 상황입니다.
  • 워크플로우 구동에 있어 권한 차이가 있을 수 있습니다. (org vs. user, upstream repo vs. forked repo)

@pbcccbeatboard-strato
Copy link
Contributor

main에서 테스트한 내용입니다.
Build and publish 부분에서 target 이슈가 있습니다.

prod로 진행
image
혹시 몰라 dev로 진행
image
동일한 내용이라 target삭제 시
image

좀 더 확인하여 테스트해본 후 다시한번 문의 드리겠습니다.

@yunkon-kim
Copy link
Member

yunkon-kim commented Aug 23, 2024

@pbcccbeatboard-strato @seokho-son (cc. @sykim-etri)

테스트 감사합니다 :-)

몇 가지 공유드릴 내용이 있습니다. 제가 Java로 구현된 App을 컨테이너화 해본 경험이 없어서 이를 감안하여 살펴봐 주시면 좋겠습니다.

  1. target: prod in Build and publish step
    : 이는 Dockerfile과 관련이 있습니다. FROM openjdk:17-slimFROM openjdk:17-slim AS prod로 수정해 주시면 될 것 같습니다.

(부연 설명)
: 저희 커뮤니티에서 Go 언어를 사용하는 프로젝트의 경우 Go Build stage와 Application Setup stage를 통해 컨테이너 이미지를 생성하고 있는 것과 관련 있습니다.
: Build 과정이 Build 환경에 영향 받는 것을 최소화 하면서, 배포하는 컨테이너 용량을 줄이기 위해 이렇게 진행되고 있습니다.
: Java의 경우에도 타당한지 살펴봐 주시면 좋을 것 같습니다.

  1. CR_PAT permission scope 관련 이슈
    : 추측하건데 permission scope 이슈가 있을 수 있는 것 같습니다.
    : 아마도 @seokho-son 께서 등록해주신 것 같은데요. 확인 부탁드립니다.

@pbcccbeatboard-strato
Copy link
Contributor

감사합니다. 작성해주신 내용 참고하여 이것저것 더 진행해보도록 하겠습니다.

@seokho-son
Copy link
Member Author

CR_PAT 는 혹시 모르니 추가 확인 및 갱신해두도록 하겠습니다. :)

@pbcccbeatboard-strato
Copy link
Contributor

말씀하신대로 AS prod로 진행하여 작동확인하였습니다.
다만 뒷부분이 아직 이해되지 않는내용이 있어 몇가지 좀 더 확인 겸 테스트 진행하도록 하겠습니다.

감사합니다. FROM 절의 AS부분은 모르고있었네요

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

No branches or pull requests

4 participants