Skip to content

K8s: add let's encrypt limitation to cert-manager page#3472

Open
kaitlynmichael wants to merge 1 commit into
mainfrom
DOC-6737
Open

K8s: add let's encrypt limitation to cert-manager page#3472
kaitlynmichael wants to merge 1 commit into
mainfrom
DOC-6737

Conversation

@kaitlynmichael

@kaitlynmichael kaitlynmichael commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

DOC-6737


Note

Low Risk
Documentation-only changes to cert-manager guidance; no runtime or security code paths modified.

Overview
Documents that Let's Encrypt and other ACME cert-manager issuers are not supported for the Redis Enterprise Kubernetes cert-manager integration, and steers readers toward issuers that supply a full chain (private CA, Vault).

The benefits list no longer mentions Let's Encrypt; it now stresses issuers that provide the root CA certificate. The former Let's Encrypt setup examples are removed and replaced with a warning: ACME issuers typically omit ca.crt in TLS secrets, while Redis Software needs the root CA to build trust—a product limitation, not Kubernetes-specific. Production guidance now points to private CA or Vault. Troubleshooting drops ACME-only bullets (DNS validation, Let's Encrypt rate limits).

The same edits apply to cert-manager.md under both the versioned (8.0.18) and unversioned Kubernetes security docs.

Reviewed by Cursor Bugbot for commit bc59224. Bugbot is set up for automated code reviews on this repo. Configure here.

@kaitlynmichael kaitlynmichael requested review from a team and heinrich-redislabs June 9, 2026 19:43
@kaitlynmichael kaitlynmichael self-assigned this Jun 9, 2026
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

DOC-6737

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

@jit-ci

jit-ci Bot commented Jun 9, 2026

Copy link
Copy Markdown

🛡️ Jit Security Scan Results

CRITICAL HIGH MEDIUM

✅ No security findings were detected in this PR


Security scan by Jit

@dwdougherty dwdougherty left a comment

Copy link
Copy Markdown
Collaborator

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants