Change secret mounting path to allign with nova-operator

openstack-k8s-operators · Feb 6, 2024 · 826dafd · 826dafd
1 parent fa32909
commit 826dafd
Show file tree

Hide file tree

Showing 5 changed files with 20 additions and 9 deletions.
diff --git a/pkg/placement/volumes.go b/pkg/placement/volumes.go
@@ -68,7 +68,7 @@ func getVolumeMounts(serviceName string) []corev1.VolumeMount {
 		},
 		{
 			Name:      "config-data",
-			MountPath: "/var/lib/config-data/merged",
+			MountPath: "/var/lib/openstack/config",
 			ReadOnly:  false,
 		},
 		{

diff --git a/templates/placementapi/config/placement-api-config.json b/templates/placementapi/config/placement-api-config.json
@@ -2,25 +2,25 @@
     "command": "/usr/sbin/httpd -DFOREGROUND",
     "config_files": [
         {
-            "source": "/var/lib/config-data/merged/placement.conf",
+            "source": "/var/lib/openstack/config/placement.conf",
             "dest": "/etc/placement/placement.conf",
             "owner": "placement",
             "perm": "0600"
         },
         {
-            "source": "/var/lib/config-data/merged/httpd.conf",
+            "source": "/var/lib/openstack/config/httpd.conf",
             "dest": "/etc/httpd/conf/httpd.conf",
             "owner": "apache",
             "perm": "0644"
         },
         {
-            "source": "/var/lib/config-data/merged/custom.conf",
+            "source": "/var/lib/openstack/config/custom.conf",
             "dest": "/etc/placement/placement.conf.d/custom.conf",
             "owner": "placement",
             "perm": "0600"
         },
         {
-            "source": "/var/lib/config-data/merged/ssl.conf",
+            "source": "/var/lib/openstack/config/ssl.conf",
             "dest": "/etc/httpd/conf.d/ssl.conf",
             "owner": "apache",
             "perm": "0644"

diff --git a/templates/placementapi/config/placement-dbsync-config.json b/templates/placementapi/config/placement-dbsync-config.json
@@ -2,13 +2,13 @@
     "command": "placement-manage db sync",
     "config_files": [
         {
-            "source": "/var/lib/config-data/merged/placement.conf",
+            "source": "/var/lib/openstack/config/placement.conf",
             "dest": "/etc/placement/placement.conf",
             "owner": "placement",
             "perm": "0600"
         },
         {
-            "source": "/var/lib/config-data/merged/custom.conf",
+            "source": "/var/lib/openstack/config/custom.conf",
             "dest": "/etc/placement/placement.conf.d/custom.conf",
             "owner": "placement",
             "perm": "0600"

diff --git a/tests/kuttl/common/assert_sample_deployment.yaml b/tests/kuttl/common/assert_sample_deployment.yaml
@@ -144,7 +144,7 @@ spec:
           readOnly: true
         - mountPath: /var/log/placement
           name: logs
-        - mountPath: /var/lib/config-data/merged
+        - mountPath: /var/lib/openstack/config
           name: config-data
         - mountPath: /var/lib/kolla/config_files/config.json
           name: config-data
@@ -186,6 +186,17 @@ status:
   availableReplicas: 1
   replicas: 1
 ---
+# the openshift annotations can't be checked through the deployment above
+apiVersion: v1
+kind: Pod
+metadata:
+  annotations:
+    openshift.io/scc: anyuid
+  labels:
+    service: placement
+status:
+  phase: Running
+---
 apiVersion: v1
 kind: Service
 metadata:

diff --git a/tests/kuttl/tests/placement_deploy_tls/03-assert.yaml b/tests/kuttl/tests/placement_deploy_tls/03-assert.yaml
@@ -151,7 +151,7 @@ spec:
           readOnly: true
         - mountPath: /var/log/placement
           name: logs
-        - mountPath: /var/lib/config-data/merged
+        - mountPath: /var/lib/openstack/config
           name: config-data
         - mountPath: /var/lib/kolla/config_files/config.json
           name: config-data