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] Docker 기반 ci/cd 구축 #10

Merged
merged 56 commits into from
Jun 27, 2024
Merged

[feat] Docker 기반 ci/cd 구축 #10

merged 56 commits into from
Jun 27, 2024

Conversation

sansan20535
Copy link
Contributor

@sansan20535 sansan20535 commented Jun 27, 2024

변경사항

CI/CD

on:
pull_request:
branches: [ "develop" ]

  • CI/CD를 develop에 pull_request할 때 실행하도록 변경했습니다.

- name: docker image 빌드 및 푸시
uses: docker/build-push-action@v4
with:
context: .
file: ./Dockerfile-dev
push: true
tags: offroadserver/offroad

  • Dockerfile을 -dev, -local로 분리하였습니다.

고려사항

  • 오류가 발생할 때마다 yaml파일이 생성되지 않아서 발생하는 경우가 대부분이었습니다. (db관련 문제도 결국 yaml이 없어서 발생)

  • dev파일에서 ${}로 중요 변수를 설정했고 이를 github action에 넣었는데 yaml파일이 생성되지 않아, github action에는 중요변수가 아닌 실제 주소와 이름, 비밀번호를 넣어서 작동시켰습니다.

Comment

  • 여러번 action을 실행한 결과 잘 되는 것을 확인했습니다.

Test

image

질문사항

  • CI_APPLICATION_DEV, CD_APPLICATION_DEV의 내용이 같아서 하나의 파일로 합치려고 했으나 CI/CD가 잘 작동하지 않아 이후 리팩토링 때 다시 시도해보도록 하겠습니다.

  • 처음pull_request에서는 dev-CD 과정에서 2번의 오류가 발생했습니다. 하지만 이와 관련하여 찾아보고 조언을 구했을 때 local-CD와 함께 진행하여서 image나 container가 중복되어서 발생한다고 들었고 재시도를 한 결과 통과하는 모습을 볼 수 있었습니다.

image image

@sansan20535 sansan20535 added Feature New feature or request Setting about setting labels Jun 27, 2024
@sansan20535 sansan20535 self-assigned this Jun 27, 2024
@sansan20535 sansan20535 requested a review from a team as a code owner June 27, 2024 03:17
Copy link
Contributor

@unanchoi unanchoi left a comment

Choose a reason for hiding this comment

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

고생 많았습니다 ~!

local-CD, local-CI 는 없어도 괜찮을 것 같아요 local에서는 CI, CD 가 없어도 되기 떄문에 !!

커밋 정리가 안되어 있는 것 같아서 파일만 없애면 제가 squash-merge 할게요!

@jun3327 리뷰 부탁드려요 ~

@sansan20535
Copy link
Contributor Author

삭제했습니다! 감사합니다 : )

Copy link
Contributor

@jun3327 jun3327 left a comment

Choose a reason for hiding this comment

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

구현하시느라 고생하셨습니다!! Dockerfile-local 부분에 대해 코멘트 달아놨습니다!

남겨주신 질문 사항에 대해서
p2;
CI_APPLICATION_DEV, CD_APPLICATION_DEV를 하나의 파일로 합칠 필요가 있을까요?? 두 과정의 구분을 위해서 파일은 그대로 남겨놔도 되지 않을까 의견드려봅니다!

Dockerfile-local Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

p2;

local용 Dockerfile은 없어도 될 것 같은데 확인 부탁드립니다!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

확인했습니다!! Dockerfile-local 삭제하고 CD 정상작동 확인했습니다!! CI, CD 파일을 말씀하신 대로 분리 했다가, 나중에 합쳐도 문제가 없다면 리팩토링 하겠습니다!!

@unanchoi unanchoi merged commit fe37084 into develop Jun 27, 2024
3 checks passed
@unanchoi unanchoi deleted the task/CICD branch June 27, 2024 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature or request Setting about setting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants