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

백엔드 CD 돌아가지 않는 오류 해결 #472

Merged
merged 3 commits into from
Aug 19, 2024

Conversation

HoeSeong123
Copy link
Contributor

관련 이슈

close #471

📍주요 변경사항

  • dev/be 브랜치로 머지되는 경우 Develop Actions 실행
  • main 브랜치로 머지되는 경우 Production Actions 실행
  • 브랜치별 실행되는 파일이 다르므로 브랜치별 개발환경 확인 로직을 제거함
  • spring 서버 도커화로 인한 배포로직 변경

🎸기타

  • 현재 프로덕션 서버와 개발 서버의 구조가 달라서 cd 파일을 분리해놨습니다.
  • 추후에 개발 서버의 구조를 프로덕션 서버와 일치시키고 cd 파일도 하나로 합칠 예정입니다.

@HoeSeong123 HoeSeong123 added bug 개발자가 의도하지 않은 상황 zap 리뷰 우선순위가 높은 사항 BE 백엔드 labels Aug 18, 2024
@HoeSeong123 HoeSeong123 added this to the 4차 스프린트 🐔 milestone Aug 18, 2024
@HoeSeong123 HoeSeong123 self-assigned this Aug 18, 2024
Copy link
Contributor

@zangsu zangsu left a comment

Choose a reason for hiding this comment

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

세상에 러너 등록까지 해 두셨네요, 너무 멋있어요 초롱!
오늘 함께하지 못해서 많이 아쉽네요, 얼른 팔로우업 할게용

@HoeSeong123 HoeSeong123 reopened this Aug 19, 2024
Copy link
Contributor

@kyum-q kyum-q left a comment

Choose a reason for hiding this comment

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

초롱 짱 수고하셨습니다 ~~

Copy link
Contributor

@jminkkk jminkkk left a comment

Choose a reason for hiding this comment

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

굿 수고했어요 🍀

Copy link
Contributor

@zeus6768 zeus6768 left a comment

Choose a reason for hiding this comment

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

초롱 첫 CD 작업 고생 많았어요!!

Comment on lines +28 to +35
deploy:
needs: build
runs-on: Production
steps:
- name: 배포 스크립트 실행
run: |
cd ${{ secrets.ZAP_DIRECTORY }}
docker compose restart
Copy link
Contributor

Choose a reason for hiding this comment

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

working-directory 속성을 이용하면 더 좋을 것 같아요~!

@zeus6768 zeus6768 merged commit df82955 into woowacourse-teams:dev/be Aug 19, 2024
2 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 bug 개발자가 의도하지 않은 상황 zap 리뷰 우선순위가 높은 사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

5 participants