Skip to content

Conversation

@GoGradually
Copy link
Collaborator

변경된 점

  • rollout을 위한 헬스 체크 포트 9095로 변경

Copilot AI review requested due to automatic review settings December 31, 2025 13:08
@GoGradually GoGradually merged commit b6d2191 into master Dec 31, 2025
2 of 3 checks passed
@GoGradually GoGradually deleted the fix/health-check-포트-변경 branch December 31, 2025 13:08
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Spring Boot Actuator의 헬스 체크 엔드포인트 포트를 기존 애플리케이션 포트(8080)에서 별도의 관리 포트(9095)로 분리하는 변경사항입니다. 이는 application-prod.yml에 설정된 management.server.port: 9095와 일치하도록 Kubernetes deployment 설정을 업데이트합니다.

  • readinessProbe와 livenessProbe의 포트를 named port "http"에서 9095로 변경
  • 관리 엔드포인트와 애플리케이션 트래픽 분리를 통한 보안 및 모니터링 개선

httpGet:
path: /actuator/health/readiness
port: http
port: 9095
Copy link

Copilot AI Dec 31, 2025

Choose a reason for hiding this comment

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

헬스 체크를 위해 포트 9095를 사용하고 있지만, deployment의 ports 섹션에 9095 포트가 정의되어 있지 않습니다.

(1) 문제점: containerPort 9095가 명시적으로 선언되지 않아, 포트가 노출되지 않을 수 있으며 서비스 디스커버리나 네트워킹 정책에서 예상대로 동작하지 않을 수 있습니다.

(2) 영향: Kubernetes에서 containerPort를 명시하지 않아도 프로브가 동작할 수 있지만, 명시적으로 선언하는 것이 베스트 프랙티스이며 가독성과 유지보수성이 향상됩니다. 특히 gateway-metrics.yaml에서 이미 9095 포트를 NodePort로 노출하고 있으므로 일관성을 위해 정의가 필요합니다.

(3) 수정 제안: deployment.yaml의 ports 섹션에 9095 포트를 추가하고, 헬스 체크 프로브에서 해당 named port를 참조하도록 수정하세요. 예를 들어:

  • ports 섹션에 "name: management, containerPort: 9095" 추가
  • readinessProbe와 livenessProbe에서 "port: management" 사용

Copilot generated this review using guidance from repository custom instructions.
httpGet:
path: /actuator/health/liveness
port: http
port: 9095
Copy link

Copilot AI Dec 31, 2025

Choose a reason for hiding this comment

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

헬스 체크를 위해 포트 9095를 사용하고 있지만, deployment의 ports 섹션에 9095 포트가 정의되어 있지 않습니다.

(1) 문제점: containerPort 9095가 명시적으로 선언되지 않아, 포트가 노출되지 않을 수 있으며 서비스 디스커버리나 네트워킹 정책에서 예상대로 동작하지 않을 수 있습니다.

(2) 영향: Kubernetes에서 containerPort를 명시하지 않아도 프로브가 동작할 수 있지만, 명시적으로 선언하는 것이 베스트 프랙티스이며 가독성과 유지보수성이 향상됩니다. 특히 gateway-metrics.yaml에서 이미 9095 포트를 NodePort로 노출하고 있으므로 일관성을 위해 정의가 필요합니다.

(3) 수정 제안: deployment.yaml의 ports 섹션에 9095 포트를 추가하고, 헬스 체크 프로브에서 해당 named port를 참조하도록 수정하세요. 예를 들어:

  • ports 섹션에 "name: management, containerPort: 9095" 추가
  • readinessProbe와 livenessProbe에서 "port: management" 사용

Copilot generated this review using guidance from repository custom instructions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants