Azure / karpenter-provider-azure / 22782127308

94%

main: 94%

Build Type

Pull #1497

github

Committed by comtalyst

Commit Message

feat: add multi-subscription support and dataDiskSizeGB to AzureNodeClass

Add per-NodeClass subscription, resource group, and location overrides
so VMs can be provisioned in different Azure subscriptions from a single
controller. Implement AZClientManager for thread-safe per-subscription
Azure SDK client caching with lazy initialization.

Also add dataDiskSizeGB field to attach a Premium_LRS managed data disk
at LUN 0 for container runtime storage or scratch space.

Changes:
- Add subscriptionID, resourceGroup, location, dataDiskSizeGB to
  AzureNodeClass CRD (v1alpha1) with kubebuilder validation markers
- Add corresponding json:"-" adapter fields to AKSNodeClass (v1beta1)
- Update AKSNodeClassFromAzureNodeClass adapter to map new fields
- Implement AZClientManager with double-checked locking for per-sub
  VM, NIC, Extension, and Subnet client creation
- Wire resolveEffectiveClients() into VM provider to route VM/NIC
  creation to correct subscription/RG/location
- Add setVMPropertiesDataDisk() following existing setVMProperties* pattern
- Update CRD YAML with new field definitions
- Add unit tests for all new functionality

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

Pull Request Pull Request #1497: feat: add multi-subscription support and dataDiskSizeGB

Run Details

61 of 180 new or added lines in 5 files covered. (33.89%)

2 existing lines in 1 file now uncovered.

58835 of 62904 relevant lines covered (93.53%)

0.95 hits per line