Merge pull request #182 from cschwede/fix-defaultconfig-regression

Fix defaultConfig regression
openstack-k8s-operators · Mar 25, 2024 · e4f8d24 · e4f8d24
2 parents 00a1e4f + 560d184
commit e4f8d24
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 1 deletion.
diff --git a/config/samples/swift_v1beta1_swift.yaml b/config/samples/swift_v1beta1_swift.yaml
@@ -12,3 +12,6 @@ spec:
     replicas: 1
     passwordSelectors:
       service: SwiftPassword
+    defaultConfigOverwrite:
+      01-proxy-server.conf: |
+        # Additional proxy config
diff --git a/templates/common/ring-sync.sh b/templates/common/ring-sync.sh
@@ -15,7 +15,7 @@ for s in account-server \
     object-server \
     object-expirer \
     proxy-server; do
-    if [ -e /var/lib/config-data/default/*${s}*.conf ]; then
+    if $(ls -1 /var/lib/config-data/default/ | grep -q "${s}"); then
         [ ! -d  /etc/swift/${s}.conf.d ] && mkdir /etc/swift/${s}.conf.d
         cp -t /etc/swift/${s}.conf.d/ /var/lib/config-data/default/*${s}*.conf
     fi

diff --git a/tests/kuttl/tests/basic-deploy/01-assert-deploy-swift.yaml b/tests/kuttl/tests/basic-deploy/01-assert-deploy-swift.yaml
@@ -141,3 +141,11 @@ apiVersion: kuttl.dev/v1beta1
 kind: TestAssert
 commands:
   - script: $SWIFT_KUTTL_DIR/../common/scripts/check_ring_rebalance_output.sh
+---
+apiVersion: kuttl.dev/v1beta1
+kind: TestAssert
+commands:
+  - script: |
+      podname=$(oc get pod -n $NAMESPACE -l component=swift-proxy | grep Running | cut -f 1 -d " ")
+      files=$(oc rsh -n $NAMESPACE -c proxy-server "$podname" /bin/sh -c "ls -1 /etc/swift/proxy-server.conf.d/")
+      echo "$files" | grep -ze "00-proxy-server.conf.*01-proxy-server.conf"