Merge pull request #5054 from nojnhuh/mp-tags

k8s-ci-robot · web-flow · commit dbbf6844e329 · 2024-08-08T17:23:49.000-07:00
eliminate unnecessary tag updates to VMSS resources
diff --git a/azure/scope/machinepool.go b/azure/scope/machinepool.go
@@ -210,7 +210,7 @@ func (m *MachinePoolScope) ScaleSetSpec(ctx context.Context) azure.ResourceSpecG
 		SubscriptionID:               m.SubscriptionID(),
 		HasReplicasExternallyManaged: m.HasReplicasExternallyManaged(ctx),
 		ClusterName:                  m.ClusterName(),
-		AdditionalTags:               m.AzureMachinePool.Spec.AdditionalTags,
+		AdditionalTags:               m.AdditionalTags(),
 		PlatformFaultDomainCount:     m.AzureMachinePool.Spec.PlatformFaultDomainCount,
 		ZoneBalance:                  m.AzureMachinePool.Spec.ZoneBalance,
 	}

Original file line number	Diff line number	Diff line change
`@@ -210,7 +210,7 @@ func (m *MachinePoolScope) ScaleSetSpec(ctx context.Context) azure.ResourceSpecG`
`210`	`210`	`SubscriptionID: m.SubscriptionID(),`
`211`	`211`	`HasReplicasExternallyManaged: m.HasReplicasExternallyManaged(ctx),`
`212`	`212`	`ClusterName: m.ClusterName(),`
`213`		`- AdditionalTags: m.AzureMachinePool.Spec.AdditionalTags,`
	`213`	`+ AdditionalTags: m.AdditionalTags(),`
`214`	`214`	`PlatformFaultDomainCount: m.AzureMachinePool.Spec.PlatformFaultDomainCount,`
`215`	`215`	`ZoneBalance: m.AzureMachinePool.Spec.ZoneBalance,`
`216`	`216`	`}`