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

Make CD checkout branch of PR rather than main #100

Open
seongbin9786 opened this issue Jan 11, 2022 · 2 comments
Open

Make CD checkout branch of PR rather than main #100

seongbin9786 opened this issue Jan 11, 2022 · 2 comments
Assignees
Labels
chore Changes to build Process or other utilities

Comments

@seongbin9786
Copy link
Member

변경 목적

checkout Action과는 별개의 얘기임.

현재 docker-compose 등의 외부 설정이 바뀌는 배포의 경우, main 브랜치로 checkout할 때에 확인할 수 없음. 이 특성을 매번 망각해서 발생하는 삽질을 줄이기 위해 특정 branch 이름을 가져와 명시적으로 checkout 하도록 함.

변경 적용점

아마도 GitHub Action 파일에 변경점이 생길 것으로 예상됨

@seongbin9786 seongbin9786 added the chore Changes to build Process or other utilities label Jan 11, 2022
@seongbin9786 seongbin9786 self-assigned this Jan 11, 2022
@seongbin9786
Copy link
Member Author

다음의 링크에 branch name을 얻는 방법이 있으므로 테스트 후 적용하면 될 것으로 보임
https://stackoverflow.com/questions/58033366/how-to-get-the-current-branch-within-github-actions

@seongbin9786
Copy link
Member Author

seongbin9786 commented Jan 25, 2022

이렇게 되는 경우, 서버 환경은

  1. Live Server
  2. Test Server
  3. Local

로 구성되게 된다.

Live Server는 지금과 같은 구성을 유지하되 push: branches: [ main ]에서만 빌드/배포되고,

Test Server는 현재 구성에서

  • testapi.classvar.com 으로 도메인 분리 및 개별 SSL 인증서 적용
  • Docker 이미지 이름을 classvar/classvar_backend에서 classvar/classvar_backend_test로 변경
  • pull_request: branches: [ main ]에서만 동작

따라서 PR에 대해 CI/CD가 정상적으로 동작하는 것을 확인하더라도 병합되지 않은 PR의 이미지로 Live Server가 배포되지 않게 된다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Changes to build Process or other utilities
Projects
None yet
Development

No branches or pull requests

1 participant