Skip to content

fix: 헬스체크 grep 쿼팅 버그 수정 #757

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.

Sign up for GitHub

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

Merged
Hexeong merged 4 commits into from
Jun 15, 2026
Merged

fix: 헬스체크 grep 쿼팅 버그 수정 #757

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
9f5015d
refactor: prod/stage 환경 nginx 블루/그린 배포 방식 도입
Hexeong Jun 13, 2026
032355a
fix: 리뷰 반영(management port 충돌 문제 해결, MySQL 경로 버그 문제 해결)
Hexeong Jun 13, 2026
85bec25
fix: 액츄에이터 노출 & dev/prod 스크립트 일치 반영
Hexeong Jun 15, 2026
3934f41
fix: 레거시 액추에이터 노출 포트와 일치시키기
Hexeong Jun 15, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .github/workflows/dev-cd.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -122,9 +122,9 @@ jobs:
# 1. Active 슬롯 확인 (upstream.conf 기준)
UPSTREAM_PORT=$(grep -oE "server 127\.0\.0\.1:[0-9]+" /etc/nginx/conf.d/upstream.conf 2>/dev/null | grep -oE "[0-9]+$" || echo "8081")
if [ "$UPSTREAM_PORT" = "8080" ]; then
ACTIVE_SLOT="blue"; ACTIVE_PORT=8080; NEW_SLOT="green"; NEW_PORT=8081; MANAGEMENT_PORT=9081
ACTIVE_SLOT="blue"; ACTIVE_PORT=8080; NEW_SLOT="green"; NEW_PORT=9080; MANAGEMENT_PORT=9081
else
ACTIVE_SLOT="green"; ACTIVE_PORT=8081; NEW_SLOT="blue"; NEW_PORT=8080; MANAGEMENT_PORT=9080
ACTIVE_SLOT="green"; ACTIVE_PORT=9080; NEW_SLOT="blue"; NEW_PORT=8080; MANAGEMENT_PORT=8081
fi
echo "Active: ${ACTIVE_SLOT}(${ACTIVE_PORT}) → Deploy: ${NEW_SLOT}(${NEW_PORT}), management: ${MANAGEMENT_PORT}"

Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/prod-cd.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,9 +138,9 @@ jobs:
# 1. Active 슬롯 확인 (upstream.conf 기준)
UPSTREAM_PORT=$(grep -oE "server 127\.0\.0\.1:[0-9]+" /etc/nginx/conf.d/upstream.conf 2>/dev/null | grep -oE "[0-9]+$" || echo "8081")
if [ "$UPSTREAM_PORT" = "8080" ]; then
ACTIVE_SLOT="blue"; ACTIVE_PORT=8080; NEW_SLOT="green"; NEW_PORT=8081; MANAGEMENT_PORT=9081
ACTIVE_SLOT="blue"; ACTIVE_PORT=8080; NEW_SLOT="green"; NEW_PORT=9080; MANAGEMENT_PORT=9081
else
ACTIVE_SLOT="green"; ACTIVE_PORT=8081; NEW_SLOT="blue"; NEW_PORT=8080; MANAGEMENT_PORT=9080
ACTIVE_SLOT="green"; ACTIVE_PORT=9080; NEW_SLOT="blue"; NEW_PORT=8080; MANAGEMENT_PORT=8081
Comment on lines +141 to +143

@chatgpt-codex-connector chatgpt-codex-connector Bot Jun 15, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Avoid reusing ports held by the active slot

On the first rollout from the parent version, the active container is still running until after the nginx switch, and the compose file uses network_mode: "host". If blue is active, it still owns management port 9080 while this starts green with APP_PORT=9080; if green is active, it still owns app port 8081 while this starts blue with MANAGEMENT_PORT=8081. In either current production state, the new Spring process cannot bind the port before the old slot is stopped, so the deployment fails instead of remaining zero-downtime. Keep the old app/management port pairs during migration or explicitly drain/stop the old listener before reusing those ports.

Useful? React with 👍 / 👎.

fi
echo "Active: ${ACTIVE_SLOT}(${ACTIVE_PORT}) → Deploy: ${NEW_SLOT}(${NEW_PORT}), management: ${MANAGEMENT_PORT}"

Expand Down
2 changes: 1 addition & 1 deletion docker-compose.dev.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ services:
environment:
- SPRING_PROFILES_ACTIVE=dev
- SERVER_PORT=${APP_PORT:-8080}
- MANAGEMENT_SERVER_PORT=${MANAGEMENT_PORT:-9080}
- MANAGEMENT_SERVER_PORT=${MANAGEMENT_PORT:-8081}
- AWS_REGION=ap-northeast-2
- SPRING_DATA_REDIS_HOST=127.0.0.1
- SPRING_DATA_REDIS_PORT=6379
Expand Down
2 changes: 1 addition & 1 deletion docker-compose.prod.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ services:
environment:
- SPRING_PROFILES_ACTIVE=prod
- SERVER_PORT=${APP_PORT:-8080}
- MANAGEMENT_SERVER_PORT=${MANAGEMENT_PORT:-9080}
- MANAGEMENT_SERVER_PORT=${MANAGEMENT_PORT:-8081}
- AWS_REGION=ap-northeast-2
- SPRING_DATA_REDIS_HOST=127.0.0.1
- SPRING_DATA_REDIS_PORT=6379
Expand Down
Loading