fix: use correct health handlers#5376
fix: use correct health handlers#5376Flo4604 wants to merge 1 commit into03-18-fix_retry_redis_conn_in_the_bg_and_fail_open_otherwisefrom
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
c5c3602 to
39a64e3
Compare
d62a21e to
3f550d3
Compare
39a64e3 to
3cf0336
Compare
3cf0336 to
21ef251
Compare
3f550d3 to
3bdc149
Compare
21ef251 to
c216917
Compare
3bdc149 to
60fb344
Compare
c216917 to
c2f78f0
Compare
60fb344 to
9763159
Compare
c2f78f0 to
3f823a1
Compare
9763159 to
4cbd947
Compare
3f823a1 to
f6c9de1
Compare
4cbd947 to
a5f4056
Compare
a5f4056 to
e64329e
Compare
f6c9de1 to
46922be
Compare
46922be to
b0443f3
Compare
e64329e to
0ddba89
Compare
b0443f3 to
d142bd7
Compare
d142bd7 to
e95f096
Compare
7be0117 to
832fd10
Compare
832fd10 to
a777cd3
Compare
e95f096 to
01d783e
Compare
a777cd3 to
c8d828b
Compare
01d783e to
9cdf0ef
Compare
9cdf0ef to
bba35ea
Compare
c8d828b to
8912607
Compare
8912607 to
8339690
Compare
bba35ea to
810a4f1
Compare

What does this PR do?
Replaces the basic health check endpoint with separate liveness and readiness probes for the sentinel service. The liveness probe continues to use the simple health endpoint, while the readiness probe now includes database connectivity and middleware engine status checks.
Adds a
PingContextmethod to the database replica to support connection health verification.Updates Kubernetes probe configurations to use distinct endpoints:
/_unkey/internal/health/livefor liveness and/_unkey/internal/health/readyfor readiness.Fixes #5367
Type of change
How should this be tested?
/_unkey/internal/health/livereturns 200 OK/_unkey/internal/health/readyreturns 200 OK when database and engine are healthyChecklist
Required
pnpm buildpnpm fmtmake fmton/godirectoryconsole.logsgit pull origin mainAppreciated