Skip to content

Commit

Permalink
Revert "set amc ready only when amcp endpoint populated"
Browse files Browse the repository at this point in the history
This reverts commit 4e1f61d.
  • Loading branch information
alexeldeib committed Aug 6, 2021
1 parent 86b91c4 commit f839fc4
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions exp/controllers/azuremanagedcluster_controller.go
Original file line number Diff line number Diff line change
Expand Up @@ -153,8 +153,9 @@ func (r *AzureManagedClusterReconciler) Reconcile(ctx context.Context, req ctrl.
return reconcile.Result{}, errors.Wrap(err, "failed to init patch helper")
}

// enqueue requests from control plane to infra cluster to keep control plane endpoint accurate.
aksCluster.Status.Ready = controlPlane.Spec.ControlPlaneEndpoint.Host != ""
// Infrastructure must be ready before control plane. We should also enqueue
// requests from control plane to infra cluster to keep control plane endpoint accurate.
aksCluster.Status.Ready = true
aksCluster.Spec.ControlPlaneEndpoint = controlPlane.Spec.ControlPlaneEndpoint

if err := patchhelper.Patch(ctx, aksCluster); err != nil {
Expand Down

0 comments on commit f839fc4

Please sign in to comment.