Azure / karpenter-provider-azure / 22828425830

94%

main: 94%

Build Type

Pull #1497

github

Committed by comtalyst

Commit Message

fix: harden multi-subscription error handling and document known limitations

- Remove //nolint:unused from resolveEffectiveClients (it is used)
- Return clear error when subscription differs but AZClientManager is nil,
  instead of silently falling through to default-subscription clients
- Add TODO on beginLaunchInstance documenting deferred multi-sub wiring
- Document known limitations in design doc (create/list/get/delete paths)
- Rewrite tests to actually exercise resolveEffectiveClients error path
- Add test for same-subscription override (no error expected)

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

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

Run Details

52 of 152 new or added lines in 6 files covered. (34.21%)

2 existing lines in 1 file now uncovered.

58814 of 62831 relevant lines covered (93.61%)

0.95 hits per line