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

release / dev 서버 분리 작업 #171

Merged
merged 20 commits into from
Sep 14, 2024
Merged

release / dev 서버 분리 작업 #171

merged 20 commits into from
Sep 14, 2024

Conversation

jjaegii
Copy link
Contributor

@jjaegii jjaegii commented Sep 13, 2024

#️⃣연관된 이슈

#169

📝작업 내용

  • GitOps 형식에 맞게 github actions + argoCD로 변경하였습니다.
  • release와 dev 브랜치 분할 배포하였습니다.
  • 그에 맞게 각각 release와 dev에 맞게 DB와 S3 서버를 연동하였습니다.

스크린샷 (선택)

스크린샷 2024-09-13 234506
image

💬리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • application.properties 내 uri 설정이 잘못되어 배포가 제대로 된 상태인지 테스트가 불가능한 상태입니다.

@jjaegii jjaegii changed the title release / dev 서버 변경 release / dev 서버 분리 작업 Sep 13, 2024
@jjaegii
Copy link
Contributor Author

jjaegii commented Sep 13, 2024

추가로 현재 argoCD 상에선 deploy 브랜치 기준으로 release와 dev의 배포가 된 상태이고,
release와 dev 브랜치에 각각 merge가 되어야 argoCD 내 배포에 매칭되는 브랜치로 변경이 가능합니다.
-> 이유로는, release, dev 브랜치에 argoCD가 배포할 수 있는 설정 파일 경로 및 yml 파일이 존재하지 않기 때문입니다.

image

Copy link
Member

@HongYeseul HongYeseul left a comment

Choose a reason for hiding this comment

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

확인 했습니다~! 👍

@HongYeseul HongYeseul merged commit 5065ae8 into release Sep 14, 2024
1 check failed
@HongYeseul HongYeseul deleted the deploy branch September 15, 2024 12:40
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.

2 participants