Azure / karpenter-provider-azure / 22691968188

94%

main: 94%

Build Type

Pull #1473

github

Committed by comtalyst

Commit Message

feat: integrate LIST-based machine cache and increase poll interval

Wire the machineListCache into DefaultAKSMachineProvider to reduce
GET Machine API calls at scale. Key changes:

- Get() checks LIST cache before hitting API, converting O(N) individual
  GETs (from drift checks, reconciliation) into O(1) cached lookups
- List() populates the cache so subsequent Get() calls are served locally
- Create/Update/Delete invalidate affected cache entries to prevent stale data
- Increase GET poll interval from 5s to 15s (reduces 200 GET/s to ~67 GET/s
  at 1000 nodes; Telescope data shows GET 429s last 17-29s each)
- Add InvalidateMachineCache() for test scenarios where conditions change
  between List() and Get() calls
- Update cloudprovider integration tests to account for cache behavior:
  GET call count assertions use BeNumerically("<=", 1) and error injection
  tests invalidate cache before setting mock errors

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

Pull Request Pull Request #1473: perf: wire LIST cache into AKS machine Get/List and increase poll interval to 15s

Run Details

41 of 41 new or added lines in 1 file covered. (100.0%)

81 existing lines in 1 file now uncovered.

59276 of 62944 relevant lines covered (94.17%)

0.96 hits per line