Skip to content

Document updating the trust bundle ConfigMap when rotating certs via kubectl#5249

Open
MyMirelHub wants to merge 2 commits into
dapr:v1.18from
MyMirelHub:docs-mtls-kubectl-configmap
Open

Document updating the trust bundle ConfigMap when rotating certs via kubectl#5249
MyMirelHub wants to merge 2 commits into
dapr:v1.18from
MyMirelHub:docs-mtls-kubectl-configmap

Conversation

@MyMirelHub

@MyMirelHub MyMirelHub commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Description

The kubectl certificate-update steps under "Custom certificates (bring your own)" only update the dapr-trust-bundle Secret. The trust bundle also lives in a dapr-trust-bundle ConfigMap (public root cert), and Sentry validates the two as a pair: if the ConfigMap's ca.crt doesn't match the Secret's, Sentry discards the bundle and silently regenerates a self-signed 1-year CA, overwriting the user's certificates. This also breaks the section's zero-downtime promise whenever the root cert changes — even when signed with the same private key.

This PR restructures that passage into numbered steps (matching the sibling self-signed subsection):

  1. Update the Secret (existing text, unchanged)
  2. If the root cert changed, sync the same root into the ConfigMap (plain PEM, not base64) — with the command; issuer-only rotations under the existing root can skip this
  3. A verification line after the Sentry restart with the exact log message that indicates the mismatch (Root and issuer certs not found: generating self signed CA), since the failure is otherwise silent

Only the manual-kubectl path is affected; Helm and dapr mtls renew-certificate update both objects atomically.

Issue reference

Fixes #5248

Checklist

@MyMirelHub
MyMirelHub force-pushed the docs-mtls-kubectl-configmap branch from eb6a58a to 11f673d Compare July 15, 2026 11:41
Signed-off-by: MyMirelHub <15373565+MyMirelHub@users.noreply.github.com>
@MyMirelHub
MyMirelHub force-pushed the docs-mtls-kubectl-configmap branch from 11f673d to ce479af Compare July 15, 2026 11:46
@MyMirelHub
MyMirelHub marked this pull request as ready for review July 15, 2026 11:47
@MyMirelHub
MyMirelHub requested review from a team as code owners July 15, 2026 11:47

@marcduiker marcduiker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

mTLS kubectl cert-update steps omit the dapr-trust-bundle ConfigMap

2 participants