Skip to content

fix(admin-token-issuer-proxy): separate readiness from liveness - #1237

Draft
mikeyrcamp wants to merge 1 commit into
mainfrom
mcamp/fix-admin-issuer-readiness
Draft

fix(admin-token-issuer-proxy): separate readiness from liveness#1237
mikeyrcamp wants to merge 1 commit into
mainfrom
mcamp/fix-admin-issuer-readiness

Conversation

@mikeyrcamp

Copy link
Copy Markdown
Contributor

What

  • use the proxy's dependency-aware /readyz endpoint for Kubernetes readiness
  • keep /healthz as the process liveness endpoint

Why

The admin token issuer proxy can start before API Keys has completed a cold
start. A compatible application image keeps the process live while retrying
transient metadata failures, but it must not receive traffic until that
metadata is initialized.

Related to #1229.

Validation

  • focused helm template assertion confirms readiness renders /readyz
  • helm lint deploy/helm/admin-token-issuer-proxy/chart ...
  • tools/ci/check-helm-charts (14 charts linted and rendered)

Dependency

This PR must remain draft until a compatible application image containing the
/readyz endpoint is released and the default image tag is updated. The
application change is prepared, but its owning source repository is currently
archived.

No third-party dependencies are added.

Signed-off-by: Mike Camp <mcamp@nvidia.com>
@coderabbitai

coderabbitai Bot commented Aug 26, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Comment @coderabbitai help to get the list of available commands.

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