Skip to content

Commit 9255571

Browse files
authored
Merge pull request #7584 from comtalyst/comtalyst/spot-refresh-1.27
[cluster-autoscaler-release-1.27] fix: correctly set the default refresh period for VMSS size (used for Spot instances)
2 parents 0c597db + f5de3e4 commit 9255571

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

cluster-autoscaler/cloudprovider/azure/azure_manager_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1033,7 +1033,7 @@ func TestGetFilteredAutoscalingGroupsVmss(t *testing.T) {
10331033
enableForceDelete: manager.config.EnableForceDelete,
10341034
curSize: 3,
10351035
sizeRefreshPeriod: manager.azureCache.refreshInterval,
1036-
getVmssSizeRefreshPeriod: time.Duration(manager.azureCache.refreshInterval) * time.Second,
1036+
getVmssSizeRefreshPeriod: manager.azureCache.refreshInterval,
10371037
InstanceCache: InstanceCache{instancesRefreshPeriod: defaultVmssInstancesRefreshPeriod},
10381038
}}
10391039
assert.True(t, assert.ObjectsAreEqualValues(expectedAsgs, asgs), "expected %#v, but found: %#v", expectedAsgs, asgs)

cluster-autoscaler/cloudprovider/azure/azure_scale_set.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ func NewScaleSet(spec *dynamic.NodeGroupSpec, az *AzureManager, curSize int64, d
121121
if az.config.GetVmssSizeRefreshPeriod != 0 {
122122
scaleSet.getVmssSizeRefreshPeriod = time.Duration(az.config.GetVmssSizeRefreshPeriod) * time.Second
123123
} else {
124-
scaleSet.getVmssSizeRefreshPeriod = time.Duration(az.azureCache.refreshInterval) * time.Second
124+
scaleSet.getVmssSizeRefreshPeriod = az.azureCache.refreshInterval
125125
}
126126

127127
if az.config.EnableDetailedCSEMessage {

0 commit comments

Comments
 (0)