Skip to content

Commit d6be293

Browse files
Merge pull request #102 from RadekManak/remove_fallback_bootstrap_script
OCPBUGS-29956: Remove startupScript vmextension lookup
2 parents dded9f7 + 4cf278d commit d6be293

File tree

1 file changed

+0
-18
lines changed

1 file changed

+0
-18
lines changed

pkg/cloud/azure/actuators/machine/reconciler.go

Lines changed: 0 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,6 @@ import (
4141
"github.com/openshift/machine-api-provider-azure/pkg/cloud/azure/services/networkinterfaces"
4242
"github.com/openshift/machine-api-provider-azure/pkg/cloud/azure/services/publicips"
4343
"github.com/openshift/machine-api-provider-azure/pkg/cloud/azure/services/resourceskus"
44-
"github.com/openshift/machine-api-provider-azure/pkg/cloud/azure/services/virtualmachineextensions"
4544
"github.com/openshift/machine-api-provider-azure/pkg/cloud/azure/services/virtualmachines"
4645
apicorev1 "k8s.io/api/core/v1"
4746
apierrors "k8s.io/apimachinery/pkg/api/errors"
@@ -93,7 +92,6 @@ func NewReconciler(scope *actuators.MachineScope) *Reconciler {
9392
interfaceLoadBalancersSvc: interfaceloadbalancers.NewService(scope),
9493
networkInterfacesSvc: networkinterfaces.NewService(scope),
9594
virtualMachinesSvc: virtualmachines.NewService(scope),
96-
virtualMachinesExtSvc: virtualmachineextensions.NewService(scope),
9795
publicIPSvc: publicips.NewService(scope),
9896
disksSvc: disks.NewService(scope),
9997
availabilitySetsSvc: availabilitysets.NewService(scope),
@@ -436,22 +434,6 @@ func (s *Reconciler) Exists(ctx context.Context) (bool, error) {
436434
return false, fmt.Errorf("returned incorrect vm interface: %v", err)
437435
}
438436

439-
if s.scope.MachineConfig.UserDataSecret == nil {
440-
vmExtSpec := &virtualmachineextensions.Spec{
441-
Name: "startupScript",
442-
VMName: s.scope.Name(),
443-
}
444-
445-
vmExt, err := s.virtualMachinesExtSvc.Get(ctx, vmExtSpec)
446-
if err != nil && vmExt == nil {
447-
return false, nil
448-
}
449-
450-
if err != nil {
451-
return false, fmt.Errorf("failed to get vm extension: %w", err)
452-
}
453-
}
454-
455437
switch machinev1.AzureVMState(*vm.ProvisioningState) {
456438
case machinev1.VMStateDeleting:
457439
return true, fmt.Errorf("vm for machine %s has unexpected 'Deleting' provisioning state", s.scope.Machine.GetName())

0 commit comments

Comments
 (0)