Skip to content

Commit d6a8184

Browse files
authored
Merge pull request #1492 from jackfrancis/singleplacementgroup-false
SinglePlacementGroup=false for Azure VMSS (MachinePool)
2 parents 75a3eb4 + d06e73e commit d6a8184

File tree

3 files changed

+8
-3
lines changed

3 files changed

+8
-3
lines changed

azure/converters/vmss_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,8 @@ func Test_SDKToVMSS(t *testing.T) {
5252
Location: to.StringPtr("westus2"),
5353
Tags: tags,
5454
VirtualMachineScaleSetProperties: &compute.VirtualMachineScaleSetProperties{
55-
ProvisioningState: to.StringPtr(string(compute.ProvisioningState1Succeeded)),
55+
SinglePlacementGroup: to.BoolPtr(false),
56+
ProvisioningState: to.StringPtr(string(compute.ProvisioningState1Succeeded)),
5657
},
5758
},
5859
[]compute.VirtualMachineScaleSetVM{

azure/services/scalesets/scalesets.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -387,6 +387,7 @@ func (s *Service) buildVMSSFromSpec(ctx context.Context, vmssSpec azure.ScaleSet
387387
Zones: to.StringSlicePtr(vmssSpec.FailureDomains),
388388
Plan: s.generateImagePlan(),
389389
VirtualMachineScaleSetProperties: &compute.VirtualMachineScaleSetProperties{
390+
SinglePlacementGroup: to.BoolPtr(false),
390391
UpgradePolicy: &compute.UpgradePolicy{
391392
Mode: compute.UpgradeModeManual,
392393
},

azure/services/scalesets/scalesets_test.go

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -150,7 +150,8 @@ func TestGetExistingVMSS(t *testing.T) {
150150
Name: to.StringPtr("Standard_D2"),
151151
},
152152
VirtualMachineScaleSetProperties: &compute.VirtualMachineScaleSetProperties{
153-
ProvisioningState: to.StringPtr("Succeeded"),
153+
SinglePlacementGroup: to.BoolPtr(false),
154+
ProvisioningState: to.StringPtr("Succeeded"),
154155
},
155156
Zones: &[]string{"1", "3"},
156157
}, nil)
@@ -182,7 +183,8 @@ func TestGetExistingVMSS(t *testing.T) {
182183
ID: to.StringPtr("my-id"),
183184
Name: to.StringPtr("my-vmss"),
184185
VirtualMachineScaleSetProperties: &compute.VirtualMachineScaleSetProperties{
185-
ProvisioningState: to.StringPtr("Succeeded"),
186+
SinglePlacementGroup: to.BoolPtr(false),
187+
ProvisioningState: to.StringPtr("Succeeded"),
186188
},
187189
}, nil)
188190
m.ListInstances(gomockinternal.AContext(), "my-rg", "my-vmss").Return([]compute.VirtualMachineScaleSetVM{}, autorest.NewErrorWithResponse("", "", &http.Response{StatusCode: 404}, "Not found"))
@@ -853,6 +855,7 @@ func newDefaultVMSS() compute.VirtualMachineScaleSet {
853855
},
854856
Zones: &[]string{"1", "3"},
855857
VirtualMachineScaleSetProperties: &compute.VirtualMachineScaleSetProperties{
858+
SinglePlacementGroup: to.BoolPtr(false),
856859
UpgradePolicy: &compute.UpgradePolicy{
857860
Mode: compute.UpgradeModeManual,
858861
},

0 commit comments

Comments
 (0)