Azure / karpenter-provider-azure / 23030618179

94%

main: 94%

Build Type

Pull #1515

github

Committed by comtalyst

Commit Message

fix: preserve primary machine's tags and zones in batch PUT body

The batch coordinator was stripping per-machine fields (tags, zones) from
the PUT body for ALL machines, including the primary. The RP's
PrepareBatchMachines creates the primary from the body and only applies
header fields to non-primary clones. The primary's header entry was
silently discarded by cleanUpInvalidDuplicateMachinesAndZones (which
pre-seeds "seen" with the PUT URL machine name).

Result: in every batch, the first machine lost its ownership tag
(karpenter.azure.com_aksmachine_nodeclaim), all custom tags, and its
zone assignment. This broke garbage collection, crash-recovery, and
zone topology spread for the primary machine.

Fix: send the primary's original template as the PUT body (tags/zones
intact) and exclude the primary from the BatchPutMachine header's
batchMachines array. Non-primary machines continue to get their
per-machine fields from the header.

Refs: Azure/karpenter-poc#1028

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

Pull Request Pull Request #1515: fix: preserve primary machine tags and zones in batch PUT body

Run Details

18 of 41 new or added lines in 2 files covered. (43.9%)

3 existing lines in 2 files now uncovered.

59141 of 63170 relevant lines covered (93.62%)

0.95 hits per line