Repository navigation
fix: serve a static /healthz for the platform startup/liveness probes - #2
Merged
Merged
Conversation
appthrust/platform#5029 (2026-10-11) made the nextjs-postgres template probe GET /healthz for startup and liveness, based on appthrust/sample-nextjs#2, but apps are generated from this template, which had no such route. Every app created from it since then was killed on its startup-probe 404 every two minutes and never became Ready (appthrust.ink facility-inspection-map-2, 2026-10-18). /healthz is static and database-independent on purpose.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
appthrust/platform#5029 (2026-10-11) set the
nextjs-postgrestemplate's probes toGET /healthz(startup, liveness) andGET /(readiness), takingappthrust/sample-nextjs#2as the reference implementation. New apps, however, are generated from this template repository, which had no/healthzroute. Result on appthrust.ink (facility-inspection-map-2, 2026-10-18): the container starts (✓ Ready in 0ms), the kubelet reportsStartup probe failed: HTTP probe failed with statuscode: 404and restarts it every two minutes; the Knative revision never becomes Ready and the platform release fails withKnativeReadinessDeadlineExceeded. Every app created from this template since 2026-10-11 is affected.Change
app/healthz/route.ts: static200 ok(force-static, no database access) — identical to sample-nextjs./healthzmust stay static and database-independent).Verification
npm ci && npm run lint && npm run buildon the branch; build output lists└ ○ /healthz(Static).