Skip to content

Increase duration of CA certificate#70

Merged
DTLP merged 1 commit intomainfrom
dl-certs
Jan 10, 2025
Merged

Increase duration of CA certificate#70
DTLP merged 1 commit intomainfrom
dl-certs

Conversation

@DTLP
Copy link
Contributor

@DTLP DTLP commented Jan 10, 2025

This is a long shot, but perhaps it helps resolving the issue of the
secret not being updated when cert-manager renews a certificate. The
idea is taken from:
cert-manager/cert-manager#5851 (comment)

This is a long shot, but perhaps it helps resolving the issue of the
secret not being updated when cert-manager renews a certificate. The
idea is taken from:
cert-manager/cert-manager#5851 (comment)
@DTLP DTLP requested a review from a team as a code owner January 10, 2025 15:23
@github-actions
Copy link
Contributor

Post kustomize build diff:

diff --git a/root-manifests/cockroachdb/manifests-cert-manager/manifests.yaml b/built-manifests/cockroachdb/manifests-cert-manager/manifests.yaml
index 90ff427..365f832 100644
--- a/root-manifests/cockroachdb/manifests-cert-manager/manifests.yaml
+++ b/built-manifests/cockroachdb/manifests-cert-manager/manifests.yaml
@@ -443,11 +443,13 @@ spec:
   commonName: cockroachdb-ca
   dnsNames:
   - cockroachdb-ca
+  duration: 4320h
   isCA: true
   issuerRef:
     group: cert-manager.io
     kind: Issuer
     name: cockroachdb-selfsigned-issuer
+  renewBefore: 2160h
   secretName: cockroachdb-ca
 ---
 apiVersion: cert-manager.io/v1

=============================================
k8s objects: 0 to add, 0 to destroy
1 changed hunks

@DTLP DTLP merged commit 7829b11 into main Jan 10, 2025
2 checks passed
@DTLP DTLP deleted the dl-certs branch January 10, 2025 15:26
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