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

백엔드 프로덕션 배포 및 환경변수 등록 자동화 #584

Merged
merged 8 commits into from
Oct 1, 2024

Conversation

pricelees
Copy link
Contributor

PR의 목적이 무엇인가요?

  • 백엔드 프로덕션 배포 자동화
  • application-prod.yml 에 JWT 설정 추가

이슈 ID는 무엇인가요?

설명

  • 배포 자동화 스크립트는 develop-backend가 아니라 develop 브랜치에 merge시 적용됩니다.
  • EC2에 action runner 설치 및 기존의 배포용 쉘 스크립트를 수정했습니다.
  • application-prod.yml에 들어가는 환경 변수에 대한 처리 방법을 수정했습니다. 이 부분은 별도의 문서로 작성해서 공유할게요

질문 혹은 공유 사항 (Optional)

제 개인 EC2에서 테스트한 뒤, 우리 EC2에서 한번 더 정상 실행 테스트한뒤 올립니다.

ec2에서 변동된 스크립트를 확인하시려면 ~/deploy/deploy.sh 와 ~/deploy/update-properties.sh를 확인해주시고 질문 있으면 편하게 남겨주세요.

Copy link
Contributor

@ay-eonii ay-eonii left a comment

Choose a reason for hiding this comment

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

갓상돌
오타 수정만 하면 될 것 같아 approve 합니다~~~

Copy link
Contributor

@ksk0605 ksk0605 left a comment

Choose a reason for hiding this comment

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

고마워요 상돌!

@@ -48,7 +48,7 @@ jobs:

deploy:
needs: build
runs-on: [self-hosted, dev]
runs-on: [self-hosted, develop]
Copy link
Contributor

Choose a reason for hiding this comment

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

Aha!

@pricelees pricelees merged commit 34a15eb into develop-backend Oct 1, 2024
1 check passed
@ss0526100 ss0526100 deleted the chore/#581 branch October 17, 2024 07:15
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.

3 participants