Skip to content

Commit e9ad8c7

Browse files
oceanc80openshift-merge-bot[bot]
authored andcommitted
UPSTREAM: <carry>: Revert "UPSTREAM: <carry>: use projected volume for CAs to avoid subPath limitations"
This reverts commit 5be8f94.
1 parent 9f0dc1d commit e9ad8c7

File tree

2 files changed

+31
-25
lines changed

2 files changed

+31
-25
lines changed

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

Lines changed: 10 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,25 @@
33
value: {"name":"catalogserver-certs", "secret":{"optional":false,"secretName":"catalogserver-cert"}}
44
- op: add
55
path: /spec/template/spec/volumes/-
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"}]}}]}}
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"}]}}
710
- op: add
811
path: /spec/template/spec/containers/1/volumeMounts/-
912
value: {"name":"catalogserver-certs", "mountPath":"/var/certs"}
1013
- op: add
1114
path: /spec/template/spec/containers/1/volumeMounts/-
12-
value: {"name":"ca-certs", "mountPath":"/var/ca-certs", "readOnly": true}
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"}
1319
- op: add
1420
path: /spec/template/spec/containers/1/args/-
1521
value: "--tls-cert=/var/certs/tls.crt"
1622
- op: add
1723
path: /spec/template/spec/containers/1/args/-
1824
value: "--tls-key=/var/certs/tls.key"
1925
- op: add
20-
path: /spec/template/spec/containers/1/env
21-
value: [{"name":"SSL_CERT_DIR", "value":"/var/ca-certs"}]
26+
path: /spec/template/spec/containers/1/args/-
27+
value: "--ca-certs-dir=/var/trusted-cas"

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,13 +68,11 @@ 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
7172
- --v=${LOG_VERBOSITY}
7273
- --global-pull-secret=openshift-config/pull-secret
7374
command:
7475
- ./manager
75-
env:
76-
- name: SSL_CERT_DIR
77-
value: /var/ca-certs
7876
image: ${CATALOGD_IMAGE}
7977
imagePullPolicy: IfNotPresent
8078
livenessProbe:
@@ -105,9 +103,12 @@ spec:
105103
name: cache
106104
- mountPath: /var/certs
107105
name: catalogserver-certs
108-
- mountPath: /var/ca-certs
109-
name: ca-certs
110-
readOnly: true
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
111112
- mountPath: /etc/containers
112113
name: etc-containers
113114
readOnly: true
@@ -139,21 +140,20 @@ spec:
139140
secret:
140141
optional: false
141142
secretName: catalogserver-cert
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
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
157157
- hostPath:
158158
path: /etc/containers
159159
type: Directory

0 commit comments

Comments
 (0)