Skip to content

feat: add standardized healthz contract - #18

Merged
cuisongliu merged 4 commits into
mainfrom
codex/add-healthz-contract
Aug 1, 2026
Merged

cuisongliu merged 4 commits into
mainfrom
codex/add-healthz-contract

Conversation

@mlhiter

@mlhiter mlhiter commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add an unauthenticated /healthz endpoint that returns 200 with {"service":"dataflow","status":"ok"} and Cache-Control: no-store.
  • Keep /health as a legacy compatibility endpoint.
  • Move the Helm startup, liveness, and readiness probes, plus the Sealos app health annotation, to /healthz.
  • Update operational and integration docs to use the stable health check path.

Verification

  • go test ./...
  • go build ./...
  • pnpm run typecheck
  • pnpm run build
  • pnpm run lint
  • helm lint deploy/charts/dataflow
  • helm template dataflow deploy/charts/dataflow --set app.enabled=true | rg -n "health-url|startupProbe|livenessProbe|readinessProbe|path: /healthz"
  • git diff --check
  • Isolated clean-worktree verification with the same patch applied to the base commit.

@cuisongliu
cuisongliu merged commit b5a4beb into main Aug 1, 2026
6 checks passed
@cuisongliu
cuisongliu deleted the codex/add-healthz-contract branch August 1, 2026 11:41
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