Skip to content

Commit 5be8f94

Browse files
UPSTREAM: <carry>: use projected volume for CAs to avoid subPath limitations
Signed-off-by: Joe Lanford <[email protected]>
1 parent 37a1fb5 commit 5be8f94

File tree

2 files changed

+25
-31
lines changed

2 files changed

+25
-31
lines changed

openshift/kustomize/overlays/openshift/olmv1-ns/patches/manager_deployment_certs.yaml

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -3,25 +3,19 @@
33
value: {"name":"catalogserver-certs", "secret":{"optional":false,"secretName":"catalogserver-cert"}}
44
- op: add
55
path: /spec/template/spec/volumes/-
6-
value: {"name":"trusted-ca-bundle", "configMap":{"optional":false,"name":"trusted-ca-bundle", "items":[{"key":"ca-bundle.crt","path":"ca-bundle.crt"}]}}
7-
- op: add
8-
path: /spec/template/spec/volumes/-
9-
value: {"name":"service-ca", "configMap":{"optional":false,"name":"openshift-service-ca.crt", "items":[{"key":"service-ca.crt","path":"service-ca.crt"}]}}
6+
value: {"name":"ca-certs", "projected": {"sources":[{"configMap":{"optional":false,"name":"trusted-ca-bundle", "items":[{"key":"ca-bundle.crt","path":"ca-bundle.crt"}]}},{"configMap":{"optional":false,"name":"openshift-service-ca.crt", "items":[{"key":"service-ca.crt","path":"service-ca.crt"}]}}]}}
107
- op: add
118
path: /spec/template/spec/containers/1/volumeMounts/-
129
value: {"name":"catalogserver-certs", "mountPath":"/var/certs"}
1310
- op: add
1411
path: /spec/template/spec/containers/1/volumeMounts/-
15-
value: {"name":"trusted-ca-bundle", "mountPath":"/var/trusted-cas/ca-bundle.crt", "subPath":"ca-bundle.crt"}
16-
- op: add
17-
path: /spec/template/spec/containers/1/volumeMounts/-
18-
value: {"name":"service-ca", "mountPath":"/var/trusted-cas/service-ca.crt", "subPath":"service-ca.crt"}
12+
value: {"name":"ca-certs", "mountPath":"/var/ca-certs", "readOnly": true}
1913
- op: add
2014
path: /spec/template/spec/containers/1/args/-
2115
value: "--tls-cert=/var/certs/tls.crt"
2216
- op: add
2317
path: /spec/template/spec/containers/1/args/-
2418
value: "--tls-key=/var/certs/tls.key"
2519
- op: add
26-
path: /spec/template/spec/containers/1/args/-
27-
value: "--ca-certs-dir=/var/trusted-cas"
20+
path: /spec/template/spec/containers/1/env
21+
value: [{"name":"SSL_CERT_DIR", "value":"/var/ca-certs"}]

openshift/manifests/14-deployment-openshift-catalogd-catalogd-controller-manager.yml

Lines changed: 21 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,13 @@ spec:
6868
- --external-address=catalogd-service.openshift-catalogd.svc
6969
- --tls-cert=/var/certs/tls.crt
7070
- --tls-key=/var/certs/tls.key
71-
- --ca-certs-dir=/var/trusted-cas
7271
- --v=${LOG_VERBOSITY}
7372
- --global-pull-secret=openshift-config/pull-secret
7473
command:
7574
- ./manager
75+
env:
76+
- name: SSL_CERT_DIR
77+
value: /var/ca-certs
7678
image: ${CATALOGD_IMAGE}
7779
imagePullPolicy: IfNotPresent
7880
livenessProbe:
@@ -103,12 +105,9 @@ spec:
103105
name: cache
104106
- mountPath: /var/certs
105107
name: catalogserver-certs
106-
- mountPath: /var/trusted-cas/ca-bundle.crt
107-
name: trusted-ca-bundle
108-
subPath: ca-bundle.crt
109-
- mountPath: /var/trusted-cas/service-ca.crt
110-
name: service-ca
111-
subPath: service-ca.crt
108+
- mountPath: /var/ca-certs
109+
name: ca-certs
110+
readOnly: true
112111
- mountPath: /etc/containers
113112
name: etc-containers
114113
readOnly: true
@@ -140,20 +139,21 @@ spec:
140139
secret:
141140
optional: false
142141
secretName: catalogserver-cert
143-
- configMap:
144-
items:
145-
- key: ca-bundle.crt
146-
path: ca-bundle.crt
147-
name: catalogd-trusted-ca-bundle
148-
optional: false
149-
name: trusted-ca-bundle
150-
- configMap:
151-
items:
152-
- key: service-ca.crt
153-
path: service-ca.crt
154-
name: openshift-service-ca.crt
155-
optional: false
156-
name: service-ca
142+
- name: ca-certs
143+
projected:
144+
sources:
145+
- configMap:
146+
items:
147+
- key: ca-bundle.crt
148+
path: ca-bundle.crt
149+
name: catalogd-trusted-ca-bundle
150+
optional: false
151+
- configMap:
152+
items:
153+
- key: service-ca.crt
154+
path: service-ca.crt
155+
name: openshift-service-ca.crt
156+
optional: false
157157
- hostPath:
158158
path: /etc/containers
159159
type: Directory

0 commit comments

Comments
 (0)