Fix readiness probe to use /readyz endpoint#739
Fix readiness probe to use /readyz endpoint#739plural-copilot[bot] wants to merge 1 commit intomainfrom
Conversation
The readiness probe in the Helm chart was incorrectly configured to use /healthz instead of /readyz. The application code (cmd/agent/kubernetes.go) registers the readiness check at /readyz, so the probe must match this endpoint.
There was a problem hiding this comment.
This PR was generated by the claude agent runtime (engine: claude). Here's some useful information you might want to know to evaluate
the ai's perfomance:
| Name | Details |
|---|---|
| 💬 Prompt | Implement the minimal safe fix for the concrete readiness probe mismatch you identified.... |
| 🔗 Run history | View run history |
Greptile SummaryThis PR fixes a one-line misconfiguration in the Helm chart where Confidence Score: 5/5Safe to merge — minimal, well-targeted fix with no side effects. Single-line correction aligning the Helm chart with the application's actual endpoint registration. No logic changes, no new dependencies, and the fix is directly validated against the source code. No files require special attention.
|
| Filename | Overview |
|---|---|
| charts/deployment-operator/values.yaml | Corrects readinessProbe path from /healthz to /readyz, aligning the Helm chart with the application's actual endpoint registration |
Reviews (1): Last reviewed commit: "Fix readiness probe to use /readyz endpo..." | Re-trigger Greptile
Summary
Fixed the readiness probe configuration in the Helm chart to use the correct
/readyzendpoint instead of/healthz.Root Cause
The application code in
cmd/agent/kubernetes.goregisters the readiness check at/readyz(line 91), but the Helm chartvalues.yamlconfigured the readinessProbe to use/healthz(line 40), causing a mismatch.Changes
charts/deployment-operator/values.yamlto changereadinessProbe.httpGet.pathfrom/healthzto/readyz/healthzValidation
/readyzendpoint viamgr.AddReadyzCheck("readyz", healthz.Ping)/healthzendpoint viamgr.AddHealthzCheck("ping", emptyDiskHealthCheck)