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

Chore : RDS 전환, CD application.yml 주입 수정 #150

Merged
merged 1 commit into from
Sep 11, 2024

Conversation

drbug2000
Copy link
Collaborator

📝 요약

🔖 변경 사항

  • application.yml을 base64로 암호화해서 주입하지 않아도 됩니다.
  • RDS가 전환 되었습니다. 기존 데이터가 초기화 됩니다.
  • 외부에서 DB 접근이 불가능합니다.

CI/CD gitHub Action flow를 요구사항대로 변경하지 못했습니다.

  1. 이중 CI 문제를 해결하기 위해 on.pull request.main을 삭제했으나, 2중 CI는 해결했지만, CI가 아예 진행되지 않고, 이전 Push CI 정보를 그대로 사용하는 것을 확인했습니다. 안정성을 위해서 기존 방식을 유지하기로 했습니다.
    develop->main 으로 PR을 생성 할 시 CI action과정이 2번 표기됩니다.(실제로는 1번)
  2. [chore/**] branch를 devlop에 merge(push)시에만 deploy action이 진행되는 것으로 설정 하려 했으나, gitHub Action에서 push시에 source branch 추적 기능을 제공해 주지 않아 구현이 복잡합니다. 안정성을 위해 기존 방법을 유지합니다.
    "chore/"로 시작하는 브랜치를 "develop"에 PR을 생성하는 시점에 바로 deploy가 진행됩니다.

📌 PR 진행 시 이러한 점들을 참고해 주세요

* P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
* P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
* P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)

Copy link
Member

@hyunn522 hyunn522 left a comment

Choose a reason for hiding this comment

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

LGTM!
chore에서 develop으로 pr을 날릴 때에 deploy 설정을 해두신 건 chore 브랜치를 application.yml 파일의 수정 용도로만 사용하는 케이스 때문인가요?

@drbug2000
Copy link
Collaborator Author

LGTM! chore에서 develop으로 pr을 날릴 때에 deploy 설정을 해두신 건 chore 브랜치를 application.yml 파일의 수정 용도로만 사용하는 케이스 때문인가요?
application.yml이나 ci/cd 파일 수정 시 해당 내용이 main에 merge 되어서 반영되기까지 너무 오래 걸려서입니다.
"chore/"로 시작 하는 수정 사항들은 최대한 빠르게 배포과정을 확인해 보려고 했습니다.

@hyunn522
Copy link
Member

hyunn522 commented Sep 9, 2024

LGTM! chore에서 develop으로 pr을 날릴 때에 deploy 설정을 해두신 건 chore 브랜치를 application.yml 파일의 수정 용도로만 사용하는 케이스 때문인가요?
application.yml이나 ci/cd 파일 수정 시 해당 내용이 main에 merge 되어서 반영되기까지 너무 오래 걸려서입니다.
"chore/"로 시작 하는 수정 사항들은 최대한 빠르게 배포과정을 확인해 보려고 했습니다.

네 좋습니다 앞으로 chore 브랜치 생성 시 고려하겠습니다

@hyunn522 hyunn522 merged commit b69d8e3 into develop Sep 11, 2024
3 checks passed
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

Successfully merging this pull request may close these issues.

Chore : gitHub Action 수정 / RDS 내부망 전환
2 participants