Merge pull request #704 from mjura/az-error

Return error when updating az
rancher · Oct 31, 2024 · 1e8544c · 1e8544c
2 parents 4486c7b + 05c217f
commit 1e8544c
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/controller/aks-cluster-config-handler.go b/controller/aks-cluster-config-handler.go
@@ -934,7 +934,7 @@ func (h *Handler) updateUpstreamClusterState(ctx context.Context, config *aksv1.
 					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)
+					return config, fmt.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)