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

#64 [ENV] github action을 이용한 CI/CD 구축 #127

Merged
merged 1 commit into from
May 11, 2024
Merged

#64 [ENV] github action을 이용한 CI/CD 구축 #127

merged 1 commit into from
May 11, 2024

Conversation

hye-on
Copy link
Contributor

@hye-on hye-on commented May 8, 2024

☀️ 작업 사항

github actions를 이용한 cicd를 구축했습니다.

☀️ 관련 이슈

resolved : #64

☀️ 참고 사항

  • 무중단배포는 추후에 붙이도록 하겠습니다.
  • application-dev.yml 파일은 노션에 올려놓았습니다. (깃허브 시크릿에도 등록되어있습니다.)
  • 저번에 작성하신 CI가 갑자기 작동이 안되는 것 같은데 확인부탁드립니다! (혹시 수정하셨나요??)
    image

@hye-on hye-on added the D-2 2일 전 까지 리뷰해주세요 label May 8, 2024
@hye-on hye-on self-assigned this May 8, 2024
@hye-on hye-on changed the title ci/cd 스크립트 생성 [ENV] github action을 이용한 CI/CD 구축 May 8, 2024
@hye-on hye-on changed the title [ENV] github action을 이용한 CI/CD 구축 #64 [ENV] github action을 이용한 CI/CD 구축 May 8, 2024
@hye-on hye-on requested a review from 05AM May 8, 2024 03:05
@hye-on hye-on added the ENV 개발 환경 설정 label May 8, 2024
@05AM
Copy link
Member

05AM commented May 9, 2024

아 알림이 안와서 pr 올리신줄 몰랐네요..! ci workflow file은 고치지 않았습니다.

application.yml이 PROD로 설정되어 있어서 그럴 수도 있고,
제가 fork한 레포지터리에서 pr을 올려서 ci에 실패하는 것일 수도 있을 것 같습니다.

image

추가)

secret에 DEV와 PROD가 제대로 등록되어 있고, 새로 올린 fork 레포지터리의 pr의 CI도 제대로 동작하는 것을 보아 현재 브런치에서 혜온님이 CI workflow file을 수정하신 부분에 문제가 발생한 것일 수도 있을것 같습니다.

롤백하시는 것이 어떨까요?

05AM
05AM previously approved these changes May 9, 2024
Copy link
Member

@05AM 05AM left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@05AM 05AM dismissed their stale review May 9, 2024 15:42

ci workflow 동작 점검

@hye-on
Copy link
Contributor Author

hye-on commented May 10, 2024

혜온님이 CI workflow file을 수정하신 부분에 문제가 발생한 것일 수도 있을것 같습니다.

롤백하시는 것이 어떨까요?

제가 ci_dev.yml 파일을 수정했다는 말씀이신가요? 혹시 어디서 변경사항을 확인하셨나요?
이 pr에서는 cicd_dev.yml파일만 올렸습니다

@hye-on
Copy link
Contributor Author

hye-on commented May 10, 2024

새로 올린 fork 레포지터리의 pr의 CI도 제대로 동작하는 것을 보아

image

fork한 레포는 make application.yml 에서 파일이 안만들어져서 ci를 통과하는 것 같습니다.

@hye-on hye-on requested a review from 05AM May 10, 2024 01:54
@hye-on
Copy link
Contributor Author

hye-on commented May 10, 2024

@05AM

저번에 같이 스터디할 때 확인한 것처럼
ci 때 스크립트에는 TOGETUP_PROD_APPLICATION로 적고
시크릿은 TOGETUP_DEV_APPLICATION으로 만드셨는데

제가 cicd를 만들면서 TOGETUP_PROD_APPLICATION를 만들고 파일 경로를 수정했습니다.
(pr에도 언급했듯이 노션에 적었습니다.)
그래서 통과가 안되는 것 같아요

ci파일을 TOGETUP_DEV_APPLICATION 시크릿을 사용하게 수정해주시면 될 것 같습니다!

@05AM
Copy link
Member

05AM commented May 10, 2024

제가 ci_dev.yml 파일을 수정했다는 말씀이신가요? 혹시 어디서 변경사항을 확인하셨나요? 이 pr에서는 cicd_dev.yml파일만 올렸습니다

앗 죄송합니다. env/cd 브런치인 줄 착각해서 그랬던 것 같습니다.

@05AM
Copy link
Member

05AM commented May 10, 2024

제가 cicd를 만들면서 TOGETUP_PROD_APPLICATION를 만들고 파일 경로를 수정했습니다. (pr에도 언급했듯이 노션에 적었습니다.) 그래서 통과가 안되는 것 같아요

혹시 파일 경로를 수정했다는 것이 무슨 뜻일까요? main/resources 같은 경로를 말씀하시는 걸까요?

ci파일을 TOGETUP_DEV_APPLICATION 시크릿을 사용하게 수정해주시면 될 것 같습니다!

네, 그런데 현재 PROD 관련 시크릿이 존재하는데 왜 ci가 실패하는지가 궁금한 것 같습니다.
우선 ci 수정 브런치를 파서 버그 수정해보겠습니다!

@05AM
Copy link
Member

05AM commented May 10, 2024

cd 급하시면 미리 ci merge 이전에 merge하셔도 괜찮을 것 같습니다!

@hye-on
Copy link
Contributor Author

hye-on commented May 11, 2024

혹시 파일 경로를 수정했다는 것이 무슨 뜻일까요? main/resources 같은 경로를 말씀하시는 걸까요?
네, 그런데 현재 PROD 관련 시크릿이 존재하는데 왜 ci가 실패하는지가 궁금한 것 같습니다. 우선 ci 수정 브런치를 파서 버그 수정해보겠습니다!

네 main/resources 안에 있는 json파일들이요!
classpath:togetup-51b17-a3704fccce411.json 와 같이 classpath로 시작하는 경로는 jar안에서 찾게 됩니다.
깃에 올려놓지 않은 파일이라서 빌드할 때 없기 때문에 jar안에서 찾을 수 없습니다.
그래서 ec2에 올려놓은 파일의 경로를 통해 찾을 수 있게 PROD 시크릿을 수정했습니다!

@hye-on hye-on merged commit ddb2d39 into main May 11, 2024
1 check failed
@hye-on hye-on deleted the env/cicd branch September 4, 2024 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
D-2 2일 전 까지 리뷰해주세요 ENV 개발 환경 설정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ENV] CI/CD 환경 설정
2 participants