1
94%
main: 94%

Ran 20 Mar 2026 03:19AM UTC

Files 159

Run time 3s

Badge

Embed ▾

Committed 20 Mar 2026 03:08AM UTC coverage: 93.62% (-0.2%) from 93.787%

Job # 23327385490.1

Build Type

Pull #1549

github

Committed by

comtalyst

Commit Message

feat: add Machine API dry-run validation to ValidationReconciler

Add a MachineAPIDryRunReconciler that validates AKSNodeClass configuration
against AKS RP's real Machine API validator by sending a dry-run PUT
request (?dryRun=true). This catches validation drift between CRD-level
rules and RP-level rules without maintaining parallel validation logic.

The implementation adds:
- dryRunPolicy: Azure SDK per-call policy that appends ?dryRun=true
  to PUT requests (same pattern as spotSystemNodePolicy)
- AKSMachinesDryRunClient: separate Machines client with the dry-run
  policy, exposed via AZClient.AKSMachinesDryRunClient()
- MachineAPIDryRunReconciler: builds a template Machine from
  AKSNodeClass fields (OSDiskSizeGB, kubelet config, FIPS, subnet)
  and sends it through the dry-run client. Sets
  ConditionTypeValidationSucceeded=False on 4xx validation errors.
  Gracefully degrades when the RP endpoint isn't deployed yet.

Depends on: aks-rp PR #15118823 (PutMachine dry-run endpoint)
Related: Azure/karpenter-poc#1708

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

Pull Request Pull Request #1549: feat: add Machine API dry-run validation to ValidationReconciler

Coverage Stats

58827 of 62836 relevant lines covered (93.62%)

0.95 hits per line

Azure / karpenter-provider-azure / 23327385490 / 1
94%
main: 94%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23327385490.1

Azure / karpenter-provider-azure / 23327385490 / 1 94% main: 94%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 23327385490.1

Azure / karpenter-provider-azure / 23327385490 / 1
94%
main: 94%

README BADGES
x