Azure / karpenter-provider-azure / 22813389206

94%

main: 94%

Build Type

Pull #1455

github

Committed by comtalyst

Commit Message

refactor: use whole-struct hashing with exclude list for batch grouping

Replace manual templateKey struct with a shallow-copy-and-zero approach:
copy MachineProperties, nil out per-machine and read-only fields (Tags,
ETag, ProvisioningState, ResourceID, Status), then JSON-marshal and hash.

This automatically includes any new fields the SDK adds (fail-safe).
A reflection-based guardrail test catches unaccounted-for fields.

Analysis confirmed all template fields are safe for batching:
- NodeLabels: from NodePool template, not NodeClaim-unique
- NodeTaints: always empty at creation time
- NodeInitializationTaints: from NodePool template
- NodeImageVersion, EnableEncryptionAtHost: from NodePool template

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

Pull Request Pull Request #1455: feat: AKS machine creation batching mode

Run Details

323 of 471 new or added lines in 9 files covered. (68.58%)

24 existing lines in 1 file now uncovered.

59118 of 62857 relevant lines covered (94.05%)

0.96 hits per line