• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

Azure / karpenter-provider-azure / 23327377885

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

Pull #1549

github

comtalyst
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 #1549: feat: add Machine API dry-run validation to ValidationReconciler

0 of 114 new or added lines in 3 files covered. (0.0%)

58827 of 62836 relevant lines covered (93.62%)

0.95 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

0.0
/pkg/controllers/nodeclass/status/machineapi_dryrun_validation.go


Source Not Available

STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc