refactor: wire up configure* helpers, delete launchtemplate.Provider

Replace the launchtemplate.Provider intermediate abstraction with direct
imagefamily.Resolver usage in DefaultVMProvider. The resolveBootstrapAndImageData()
bridge function builds StaticParameters locally and calls imagefamily.Resolve()
directly, eliminating the 7-layer parameter threading.

New vminstancehelpers.go provides buildVMTemplate() with focused configure*
helpers (configureStorageProfile, configureOSProfile, configureNetworkProfile,
configureBillingProfile, configureSecurityProfile) matching the AKS Machine
pattern. Extension spec builders are also extracted here.

vminstancenic.go updated to take subnetID/tags directly instead of
*launchtemplate.Template, with buildAndCreateNIC() orchestrator.

Delete pkg/providers/launchtemplate/launchtemplate.go (Provider struct).
tags.go and parameters/ remain as they are still used by imagefamily.Resolve().

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: comtalyst

pkg/operator/operator.go

@@ -63,7 +63,6 @@ import (
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/instance"
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/instancetype"
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/kubernetesversion"
-	"github.com/Azure/karpenter-provider-azure/pkg/providers/launchtemplate"
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/loadbalancer"
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/networksecuritygroup"
 	"github.com/Azure/karpenter-provider-azure/pkg/providers/pricing"
@@ -88,7 +87,6 @@ type Operator struct {
 	KubernetesVersionProvider kubernetesversion.KubernetesVersionProvider
 	ImageProvider             imagefamily.NodeImageProvider
 	ImageResolver             imagefamily.Resolver
-	LaunchTemplateProvider    *launchtemplate.Provider
 	PricingProvider           *pricing.Provider
 	InstanceTypesProvider     instancetype.Provider
 	VMInstanceProvider        *instance.DefaultVMProvider
@@ -187,20 +185,6 @@ func NewOperator(ctx context.Context, operator *operator.Operator) (context.Cont
 		instanceTypeProvider,
 		azClient.NodeBootstrappingClient,
 	)
-	launchTemplateProvider := launchtemplate.NewProvider(
-		ctx,
-		imageResolver,
-		imageProvider,
-		lo.Must(getCABundle(operator.GetConfig())),
-		options.FromContext(ctx).ClusterEndpoint,
-		azConfig.TenantID,
-		azConfig.SubscriptionID,
-		azConfig.ResourceGroup,
-		options.FromContext(ctx).KubeletIdentityClientID,
-		options.FromContext(ctx).NodeResourceGroup,
-		azConfig.Location,
-		options.FromContext(ctx).ProvisionMode,
-	)
 	loadBalancerProvider := loadbalancer.NewProvider(
 		azClient.LoadBalancersClient,
 		cache.New(loadbalancer.LoadBalancersCacheTTL, azurecache.DefaultCleanupInterval),
@@ -213,7 +197,7 @@ func NewOperator(ctx context.Context, operator *operator.Operator) (context.Cont
 	vmInstanceProvider := instance.NewDefaultVMProvider(
 		azClient,
 		instanceTypeProvider,
-		launchTemplateProvider,
+		imageResolver,
 		loadBalancerProvider,
 		networkSecurityGroupProvider,
 		unavailableOfferingsCache,
@@ -223,6 +207,10 @@ func NewOperator(ctx context.Context, operator *operator.Operator) (context.Cont
 		options.FromContext(ctx).ProvisionMode,
 		options.FromContext(ctx).DiskEncryptionSetID,
 		env,
+		lo.Must(getCABundle(operator.GetConfig())),
+		options.FromContext(ctx).ClusterEndpoint,
+		azConfig.TenantID,
+		azConfig.ResourceGroup,
 	)
 	aksMachineInstanceProvider := instance.NewAKSMachineProvider(
 		azClient,
@@ -243,7 +231,6 @@ func NewOperator(ctx context.Context, operator *operator.Operator) (context.Cont
 		KubernetesVersionProvider:    kubernetesVersionProvider,
 		ImageProvider:                imageProvider,
 		ImageResolver:                imageResolver,
-		LaunchTemplateProvider:       launchTemplateProvider,
 		PricingProvider:              pricingProvider,
 		InstanceTypesProvider:        instanceTypeProvider,
 		VMInstanceProvider:           vmInstanceProvider,