Skip to content

Commit d5abd4d

Browse files
authored
Merge pull request #674 from percona/K8SPS-310
K8SPS-310 update tls certificate issuer name
2 parents 09a2b55 + 0dd9ed0 commit d5abd4d

File tree

10 files changed

+24
-24
lines changed

10 files changed

+24
-24
lines changed

e2e-tests/tests/gr-finalizer/02-assert.yaml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ spec:
1313
isCA: true
1414
issuerRef:
1515
kind: Issuer
16-
name: gr-finalizer-pso-ca-issuer
16+
name: gr-finalizer-ps-ca-issuer
1717
renewBefore: 730h0m0s
1818
secretName: gr-finalizer-ca-cert
1919
status:
@@ -33,7 +33,7 @@ metadata:
3333
spec:
3434
issuerRef:
3535
kind: Issuer
36-
name: gr-finalizer-pso-issuer
36+
name: gr-finalizer-ps-issuer
3737
secretName: test-ssl
3838
status:
3939
conditions:
@@ -90,7 +90,7 @@ metadata:
9090
cert-manager.io/certificate-name: gr-finalizer-ca-cert
9191
cert-manager.io/common-name: gr-finalizer-ca
9292
cert-manager.io/issuer-kind: Issuer
93-
cert-manager.io/issuer-name: gr-finalizer-pso-ca-issuer
93+
cert-manager.io/issuer-name: gr-finalizer-ps-ca-issuer
9494
labels:
9595
controller.cert-manager.io/fao: "true"
9696
name: gr-finalizer-ca-cert
@@ -102,7 +102,7 @@ metadata:
102102
annotations:
103103
cert-manager.io/certificate-name: gr-finalizer-ssl
104104
cert-manager.io/issuer-kind: Issuer
105-
cert-manager.io/issuer-name: gr-finalizer-pso-issuer
105+
cert-manager.io/issuer-name: gr-finalizer-ps-issuer
106106
labels:
107107
controller.cert-manager.io/fao: "true"
108108
name: test-ssl
@@ -112,15 +112,15 @@ apiVersion: cert-manager.io/v1
112112
kind: Issuer
113113
metadata:
114114
generation: 1
115-
name: gr-finalizer-pso-ca-issuer
115+
name: gr-finalizer-ps-ca-issuer
116116
spec:
117117
selfSigned: {}
118118
---
119119
apiVersion: cert-manager.io/v1
120120
kind: Issuer
121121
metadata:
122122
generation: 1
123-
name: gr-finalizer-pso-issuer
123+
name: gr-finalizer-ps-issuer
124124
spec:
125125
ca:
126126
secretName: gr-finalizer-ca-cert

e2e-tests/tests/gr-finalizer/07-errors.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,12 @@ metadata:
3636
apiVersion: cert-manager.io/v1
3737
kind: Issuer
3838
metadata:
39-
name: gr-finalizer-pso-ca-issuer
39+
name: gr-finalizer-ps-ca-issuer
4040
---
4141
apiVersion: cert-manager.io/v1
4242
kind: Issuer
4343
metadata:
44-
name: gr-finalizer-pso-issuer
44+
name: gr-finalizer-ps-issuer
4545
---
4646
apiVersion: apps/v1
4747
kind: StatefulSet

e2e-tests/tests/gr-tls-cert-manager/02-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: gr-tls-cert-manager-pso-ca-issuer
15+
name: gr-tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: gr-tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: gr-tls-cert-manager-pso-issuer
34+
name: gr-tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

e2e-tests/tests/gr-tls-cert-manager/04-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: gr-tls-cert-manager-pso-ca-issuer
15+
name: gr-tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: gr-tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: gr-tls-cert-manager-pso-issuer
34+
name: gr-tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

e2e-tests/tests/gr-tls-cert-manager/06-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: gr-tls-cert-manager-pso-ca-issuer
15+
name: gr-tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: gr-tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: gr-tls-cert-manager-pso-issuer
34+
name: gr-tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

e2e-tests/tests/tls-cert-manager/02-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: tls-cert-manager-pso-ca-issuer
15+
name: tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: tls-cert-manager-pso-issuer
34+
name: tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

e2e-tests/tests/tls-cert-manager/04-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: tls-cert-manager-pso-ca-issuer
15+
name: tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: tls-cert-manager-pso-issuer
34+
name: tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

e2e-tests/tests/tls-cert-manager/06-assert.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ spec:
1212
isCA: true
1313
issuerRef:
1414
kind: Issuer
15-
name: tls-cert-manager-pso-ca-issuer
15+
name: tls-cert-manager-ps-ca-issuer
1616
renewBefore: 730h0m0s
1717
secretName: tls-cert-manager-ca-cert
1818
status:
@@ -31,7 +31,7 @@ metadata:
3131
spec:
3232
issuerRef:
3333
kind: Issuer
34-
name: tls-cert-manager-pso-issuer
34+
name: tls-cert-manager-ps-issuer
3535
secretName: test-ssl
3636
status:
3737
conditions:

pkg/controller/ps/controller.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -285,8 +285,8 @@ func (r *PerconaServerMySQLReconciler) deleteCerts(ctx context.Context, cr *apiv
285285
log.Info("Deleting SSL certificates")
286286

287287
issuers := []string{
288-
cr.Name + "-pso-ca-issuer",
289-
cr.Name + "-pso-issuer",
288+
cr.Name + "-ps-ca-issuer",
289+
cr.Name + "-ps-issuer",
290290
}
291291
for _, issuerName := range issuers {
292292
issuer := &cm.Issuer{}

pkg/controller/ps/tls.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -110,8 +110,8 @@ func (r *PerconaServerMySQLReconciler) checkTLSIssuer(ctx context.Context, cr *a
110110
}
111111

112112
func (r *PerconaServerMySQLReconciler) ensureSSLByCertManager(ctx context.Context, cr *apiv1alpha1.PerconaServerMySQL) error {
113-
issuerName := cr.Name + "-pso-issuer"
114-
caIssuerName := cr.Name + "-pso-ca-issuer"
113+
issuerName := cr.Name + "-ps-issuer"
114+
caIssuerName := cr.Name + "-ps-ca-issuer"
115115
issuerKind := "Issuer"
116116
issuerGroup := ""
117117
if cr.Spec.TLS != nil && cr.Spec.TLS.IssuerConf != nil {

0 commit comments

Comments
 (0)