Skip to content

Fix/헬스 체크 포트 9095로 변경 #7

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
GoGradually merged 1 commit into from
Dec 31, 2025
Merged

Fix/헬스 체크 포트 9095로 변경 #7

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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 k8s/deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ spec:
readinessProbe:
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.
initialDelaySeconds: 10
periodSeconds: 5
timeoutSeconds: 2
Expand All @@ -62,7 +62,7 @@ spec:
livenessProbe:
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.
initialDelaySeconds: 30
periodSeconds: 10
timeoutSeconds: 2
Expand Down
Loading