Fix empty vmType (#755)

gardener · Nov 23, 2023 · 21c1c67 · 21c1c67
1 parent 9d7d4ab
commit 21c1c67
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/controller/controlplane/valuesprovider.go b/pkg/controller/controlplane/valuesprovider.go
@@ -486,17 +486,16 @@ func getConfigChartValues(infraStatus *apisazure.InfrastructureStatus, cp *exten
 }
 
 func appendMachineSetValues(values map[string]interface{}, infraStatus *apisazure.InfrastructureStatus) map[string]interface{} {
+	values["vmType"] = "standard"
 	if azureapihelper.IsVmoRequired(infraStatus) {
 		values["vmType"] = "vmss"
 		return values
 	}
 
 	if primaryAvailabilitySet, err := azureapihelper.FindAvailabilitySetByPurpose(infraStatus.AvailabilitySets, apisazure.PurposeNodes); err == nil {
 		values["availabilitySetName"] = primaryAvailabilitySet.Name
-		return values
 	}
 
-	values["vmType"] = "standard"
 	return values
 }
 

diff --git a/pkg/controller/controlplane/valuesprovider_test.go b/pkg/controller/controlplane/valuesprovider_test.go
@@ -244,6 +244,7 @@ var _ = Describe("ValuesProvider", func() {
 					"routeTableName":      "route-table-name",
 					"securityGroupName":   "security-group-name-workers",
 					"maxNodes":            maxNodes,
+					"vmType":              "standard",
 				}))
 			})