Merge pull request #171 from cschwede/fix-statefulset-parallel-start

Fix regression when scaling up

pkg/swiftstorage/statefulset.go

@@ -205,7 +205,8 @@ func StatefulSet(
 			Selector: &metav1.LabelSelector{
 				MatchLabels: labels,
 			},
-			Replicas: swiftstorage.Spec.Replicas,
+			Replicas:            swiftstorage.Spec.Replicas,
+			PodManagementPolicy: appsv1.ParallelPodManagement,
 			Template: corev1.PodTemplateSpec{
 				ObjectMeta: metav1.ObjectMeta{
 					Labels:      labels,

tests/kuttl/tests/basic-deploy/02-assert-scaleup-swift.yaml

@@ -0,0 +1,21 @@
+---
+apiVersion: v1
+kind: Pod
+metadata:
+  name: swift-storage-0
+status:
+  phase: Running
+---
+apiVersion: v1
+kind: Pod
+metadata:
+  name: swift-storage-1
+status:
+  phase: Running
+---
+apiVersion: v1
+kind: Pod
+metadata:
+  name: swift-storage-2
+status:
+  phase: Running

tests/kuttl/tests/basic-deploy/02-scaleup-swift.yaml

@@ -0,0 +1,5 @@
+apiVersion: kuttl.dev/v1beta1
+kind: TestStep
+commands:
+  - script: |
+      oc patch -n swift-kuttl-tests swift/swift --type='json' -p='[{"op": "replace", "path": "/spec/swiftStorage/replicas", "value":3}]'

tests/kuttl/tests/basic-deploy/02-cleanup.yaml → tests/kuttl/tests/basic-deploy/03-cleanup.yaml

@@ -7,3 +7,5 @@ delete:
 commands:
 - script: |
     oc delete --ignore-not-found=true -n $NAMESPACE pvc srv-swift-storage-0
+    oc delete --ignore-not-found=true -n $NAMESPACE pvc srv-swift-storage-1
+    oc delete --ignore-not-found=true -n $NAMESPACE pvc srv-swift-storage-2