Skip to content

Commit 4285571

Browse files
Change secret mounting path to allign with nova-operator
1 parent 7090b23 commit 4285571

File tree

5 files changed

+20
-9
lines changed

5 files changed

+20
-9
lines changed

pkg/placement/volumes.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ func getVolumeMounts(serviceName string) []corev1.VolumeMount {
6868
},
6969
{
7070
Name: "config-data",
71-
MountPath: "/var/lib/config-data/merged",
71+
MountPath: "/var/lib/openstack/config",
7272
ReadOnly: false,
7373
},
7474
{

templates/placementapi/config/placement-api-config.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2,25 +2,25 @@
22
"command": "/usr/sbin/httpd -DFOREGROUND",
33
"config_files": [
44
{
5-
"source": "/var/lib/config-data/merged/placement.conf",
5+
"source": "/var/lib/openstack/config/placement.conf",
66
"dest": "/etc/placement/placement.conf",
77
"owner": "placement",
88
"perm": "0600"
99
},
1010
{
11-
"source": "/var/lib/config-data/merged/httpd.conf",
11+
"source": "/var/lib/openstack/config/httpd.conf",
1212
"dest": "/etc/httpd/conf/httpd.conf",
1313
"owner": "apache",
1414
"perm": "0644"
1515
},
1616
{
17-
"source": "/var/lib/config-data/merged/custom.conf",
17+
"source": "/var/lib/openstack/config/custom.conf",
1818
"dest": "/etc/placement/placement.conf.d/custom.conf",
1919
"owner": "placement",
2020
"perm": "0600"
2121
},
2222
{
23-
"source": "/var/lib/config-data/merged/ssl.conf",
23+
"source": "/var/lib/openstack/config/ssl.conf",
2424
"dest": "/etc/httpd/conf.d/ssl.conf",
2525
"owner": "apache",
2626
"perm": "0644"

templates/placementapi/config/placement-dbsync-config.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,13 @@
22
"command": "placement-manage db sync",
33
"config_files": [
44
{
5-
"source": "/var/lib/config-data/merged/placement.conf",
5+
"source": "/var/lib/openstack/config/placement.conf",
66
"dest": "/etc/placement/placement.conf",
77
"owner": "placement",
88
"perm": "0600"
99
},
1010
{
11-
"source": "/var/lib/config-data/merged/custom.conf",
11+
"source": "/var/lib/openstack/config/custom.conf",
1212
"dest": "/etc/placement/placement.conf.d/custom.conf",
1313
"owner": "placement",
1414
"perm": "0600"

tests/kuttl/common/assert_sample_deployment.yaml

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -144,7 +144,7 @@ spec:
144144
readOnly: true
145145
- mountPath: /var/log/placement
146146
name: logs
147-
- mountPath: /var/lib/config-data/merged
147+
- mountPath: /var/lib/openstack/config
148148
name: config-data
149149
- mountPath: /var/lib/kolla/config_files/config.json
150150
name: config-data
@@ -186,6 +186,17 @@ status:
186186
availableReplicas: 1
187187
replicas: 1
188188
---
189+
# the openshift annotations can't be checked through the deployment above
190+
apiVersion: v1
191+
kind: Pod
192+
metadata:
193+
annotations:
194+
openshift.io/scc: anyuid
195+
labels:
196+
service: placement
197+
status:
198+
phase: Running
199+
---
189200
apiVersion: v1
190201
kind: Service
191202
metadata:

tests/kuttl/tests/placement_deploy_tls/03-assert.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ spec:
151151
readOnly: true
152152
- mountPath: /var/log/placement
153153
name: logs
154-
- mountPath: /var/lib/config-data/merged
154+
- mountPath: /var/lib/openstack/config
155155
name: config-data
156156
- mountPath: /var/lib/kolla/config_files/config.json
157157
name: config-data

0 commit comments

Comments
 (0)