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

nginx reload 스크립트 고침 (2) #1020

Merged
merged 1 commit into from
Jan 6, 2025
Merged

nginx reload 스크립트 고침 (2) #1020

merged 1 commit into from
Jan 6, 2025

Conversation

zeus6768
Copy link
Contributor

@zeus6768 zeus6768 commented Jan 6, 2025

⚡️ 관련 이슈

#833

📍주요 변경 사항

주요 변경 사항에 대해 작성해주세요.

🎸기타

고려해야 하는 내용을 작성해 주세요.

🍗 PR 첫 리뷰 마감 기한

00/00 00:00

@zeus6768 zeus6768 added zap 리뷰 우선순위가 높은 사항 BE 백엔드 infrastructure 인프라 작업 labels Jan 6, 2025
@zeus6768 zeus6768 self-assigned this Jan 6, 2025
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.

진짜 제발~~

@jminkkk jminkkk merged commit e7fe84f into dev/be Jan 6, 2025
5 checks passed
@jminkkk jminkkk deleted the fix/cd-nginx-reload branch January 6, 2025 06:32
@zangsu
Copy link
Contributor

zangsu commented Jan 6, 2025

nginx_reload 가 잘 되는지 확인하기 위해 상태값을 확인하도록 변경

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 infrastructure 인프라 작업 zap 리뷰 우선순위가 높은 사항
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

4 participants