Wake-up-together-TogetUp / togetup-server

AI를 활용한 커뮤니티 기반 미션 알람 서비스, TogetUp!
1 stars 1 forks source link

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

Closed hye-on closed 4 months ago

hye-on commented 4 months ago

☀️ 작업 사항

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

☀️ 관련 이슈

resolved : #64

☀️ 참고 사항

05AM commented 4 months ago

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

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

image

추가)

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

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

hye-on commented 4 months ago

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

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

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

hye-on commented 4 months ago

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

image

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

hye-on commented 4 months ago

@05AM

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

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

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

05AM commented 4 months ago

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

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

05AM commented 4 months ago

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

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

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

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

05AM commented 4 months ago

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

hye-on commented 4 months ago

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

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