Repository navigation
feat(OMN-13753): provision missing prod secrets — omnidash-tls Certificate + umami-credentials InfisicalSecret - #2158
jonahgabriel wants to merge 2 commits into
Conversation
…icate + umami-credentials InfisicalSecret Adds declarative manifests for the two missing onex-prod Kubernetes secrets that are causing traefik TLS errors and the umami pod Init:CreateContainerConfigError. Changes: - deploy/k8s/onex-prod/omnidash/certificate.yaml: explicit cert-manager Certificate resource for omnidash-tls (dash.omninode.ai) via letsencrypt-prod ClusterIssuer. Explicit Certificate is operator-inspectable and force-renewable, vs relying solely on the implicit Ingress annotation path. - deploy/k8s/onex-prod/umami/infisical-secret.yaml: InfisicalSecret CRD that syncs umami-credentials from Infisical /prod/umami/ following the omnidash pattern. Includes Path B guidance for imperative provisioning via setup-cluster-secrets.sh. - contracts/OMN-13753.yaml: OCC contract with dod_evidence. - drift/dod_receipts/OMN-13753/: deploy gate + lint receipts. No literal secret values committed. Applying to the cluster = operator gate (OMN-13418).
|
Warning Review limit reachedYou’ve reached a temporary PR review limit under our Fair Usage Limits Policy. Next review available in: 27 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
….kind + drop invalid dod_evidence.status
|
Closing in favor of OmniNode-ai/omninode_infra#531. Why misplaced here: k8s manifests for onex-prod belong in The canonical fix is in the correct repo at: https://github.com/OmniNode-ai/omninode_infra/pull/531 Closing this PR. |
Pull request was closed
Summary
Fixes two missing
onex-prodKubernetes secrets that are causing live prod alerts:Error configuring TLS: secret onex-prod/omnidash-tls does not existevery ~5m →dash.omninode.aiTLS brokenInit:CreateContainerConfigError→secret "umami-credentials" not found→ analytics downChanges
deploy/k8s/onex-prod/omnidash/certificate.yaml— explicit cert-managerCertificateresource foromnidash-tls, targetingdash.omninode.aivia theletsencrypt-prodClusterIssuer (HTTP-01). The omnidashIngressalready carries thecert-manager.io/cluster-issuer: letsencrypt-prodannotation which triggers automatic issuance, but an explicitCertificategives operators a named object they candescribe, force-renew, and alert on independently of the Ingress object.deploy/k8s/onex-prod/umami/infisical-secret.yaml—InfisicalSecretCRD that syncsumami-credentialsfrom Infisical path/prod/umami/following the same pattern asomnidash-secret.yamlinomninode_infra. Includes Path B (imperative fallback) guidance for runningsetup-cluster-secrets.sh create_umami_credentialsif Infisical is not yet seeded.contracts/OMN-13753.yaml— OCC contract withdod_evidence.drift/dod_receipts/OMN-13753/— deploy gate + lint receipts.No literal secrets committed
All secret references use
secretKeyRef/credentialsRefpointing to named Kubernetes secrets — values resolve at the cluster boundary (Infisical operator orsetup-cluster-secrets.sh).Operator action required (OMN-13418 gate)
This PR delivers manifest declarations. Applying to the cluster requires a fresh operator approval:
Evidence-Source
Evidence-Source: OCC#3356
Evidence-Ticket: OMN-13753