Reduce concurrency for OS SUC workers (#107)

* Reduce concurrency for OS SUC workers * Fix unit tests
suse-edge · Oct 29, 2024 · 15fcb10 · 15fcb10
1 parent 14134a3
commit 15fcb10
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/internal/upgrade/os.go b/internal/upgrade/os.go
@@ -114,7 +114,7 @@ func OSWorkerPlan(nameSuffix, releaseVersion, secretName string, releaseOS *life
 
 	labels["os-upgrade"] = "worker"
 	workerPlan := baseOSPlan(workerPlanName, releaseVersion, secretName, drain, labels)
-	workerPlan.Spec.Concurrency = 2
+	workerPlan.Spec.Concurrency = 1
 	workerPlan.Spec.NodeSelector = &metav1.LabelSelector{
 		MatchExpressions: []metav1.LabelSelectorRequirement{
 			{

diff --git a/internal/upgrade/os_test.go b/internal/upgrade/os_test.go
@@ -158,7 +158,7 @@ func TestOSWorkerPlan(t *testing.T) {
 	assert.Equal(t, []string{"sh", "/run/system-upgrade/secrets/some-secret/os-upgrade.sh"}, upgradeContainer.Args)
 
 	assert.Equal(t, "3.1.0", upgradePlan.Spec.Version)
-	assert.EqualValues(t, 2, upgradePlan.Spec.Concurrency)
+	assert.EqualValues(t, 1, upgradePlan.Spec.Concurrency)
 	assert.EqualValues(t, 43200, upgradePlan.Spec.JobActiveDeadlineSecs)
 	assert.True(t, upgradePlan.Spec.Cordon)