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

[Fix] #218 - 개발, 운영서버 분리 #219

Merged
merged 6 commits into from
Feb 27, 2024
Merged

[Fix] #218 - 개발, 운영서버 분리 #219

merged 6 commits into from
Feb 27, 2024

Conversation

its-sky
Copy link
Member

@its-sky its-sky commented Feb 27, 2024

🚀PullRequest🚀

📟 관련 이슈

💻 작업 내용

  • 개발, 운영 서버 분리를 위한 작업을 진행하였습니다.
  • 기존 CI, CD.yml 파일을 prod-CI.yml, prod-CD.yml로 변경하고 PR 브랜치 조건을 main으로 변경하였습니다.
  • 개발서버 CI, CD.yml 파일은 branch가 develop으로 PR이 생성되고 병합될 때 실행됩니다.
  • 개발 서버 Dockerfile을 Dockerfile-dev로 생성하였고 dev-CD.yml에서 빌드 시 -f Dockerfile-dev를 통하여 해당 도커파일을 지정하도록 하였습니다.
  • 개발 서버의 Github Actions Secret 분리를 하였습니다.

📝 리뷰 노트

Copy link
Member

@0lynny 0lynny left a comment

Choose a reason for hiding this comment

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

제발 성공하길! 수고 많으셨습니당~

- name: application.yaml 작성
run: |
cd moonshot-api/src/main/resources
echo "${{ secrets.DEV-APPLICATION }}" > ./application-dev.yml
Copy link
Member

Choose a reason for hiding this comment

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

p3;
DEV_APPLICATION으로 수정 부탁드립니다!

@its-sky its-sky merged commit 72f1026 into develop Feb 27, 2024
1 check passed
@its-sky its-sky deleted the feature/#218 branch February 27, 2024 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Fix] deploy-CI,CD 변경 및 dev 서버 CI,CD.yml 추가
2 participants