Merge pull request #1767 from gvanderpotte/startVM

change order to ensure startVM is done early
Azure · Oct 1, 2021 · 880316a · 880316a
2 parents d61fd16 + d36a5bd
commit 880316a
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/pkg/cluster/install.go b/pkg/cluster/install.go
@@ -35,9 +35,9 @@ func (m *manager) AdminUpdate(ctx context.Context) error {
 		steps.Action(m.ensureDefaults),
 		steps.AuthorizationRefreshingAction(m.fpAuthorizer, steps.Action(m.ensureResourceGroup)), // re-create RP RBAC if needed after tenant migration
 		steps.Action(m.createOrUpdateDenyAssignment),
-		steps.Action(m.populateRegistryStorageAccountName),
 		steps.Action(m.startVMs),
 		steps.Condition(m.apiServersReady, 30*time.Minute, false),
+		steps.Action(m.populateRegistryStorageAccountName),
 		steps.Action(m.ensureBillingRecord), // belt and braces
 		steps.Action(m.configureAPIServerCertificate),
 		steps.Action(m.configureIngressCertificate),