Merge pull request #674 from mjura/az-v2.9

[v2.9] Add validation for modifications to availability zones
rancher · Oct 1, 2024 · fbfc9d9 · fbfc9d9
2 parents 947a102 + 8f4a309
commit fbfc9d9
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/controller/aks-cluster-config-handler.go b/controller/aks-cluster-config-handler.go
@@ -944,6 +944,9 @@ func (h *Handler) updateUpstreamClusterState(ctx context.Context, config *aksv1.
 					logrus.Debugf("config: %s; upstream: %s", np.Mode, upstreamNodePool.Mode)
 					updateNodePool = true
 				}
+				if np.AvailabilityZones != nil && !reflect.DeepEqual(np.AvailabilityZones, upstreamNodePool.AvailabilityZones) {
+					logrus.Errorf("Changing availability zones for node pool [%s] in cluster [%s (id: %s)] is not permitted", aks.String(np.Name), config.Spec.ClusterName, config.Name)
+				}
 			} else {
 				logrus.Infof("Adding node pool [%s] for cluster [%s (id: %s)]", aks.String(np.Name), config.Spec.ClusterName, config.Name)
 				updateNodePool = true