[AKS] aks-preview: Add --zones to nodepool update - #10248
[AKS] aks-preview: Add --zones to nodepool update#10248Alex Huang (wenhug) wants to merge 2 commits into
Conversation
|
Hi Alex Huang (@wenhug), |
There was a problem hiding this comment.
Pull request overview
Adds preview --zones/-z support to az aks nodepool update, including automatic zone placement and compatibility with VM-size updates.
Changes:
- Adds zone parameters, help text, and release metadata.
- Applies zone updates while preserving omitted values.
- Adds compatibility and unit test coverage; bumps version to
22.0.0b4.
Reviewed changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.
Show a summary per file
| File | Summary |
|---|---|
src/aks-preview/setup.py |
Bumps extension version. |
src/aks-preview/HISTORY.rst |
Documents the new option. |
src/aks-preview/azext_aks_preview/tests/latest/test_update_agentpool_profile_preview.py |
Tests update-flow compatibility. |
src/aks-preview/azext_aks_preview/tests/latest/test_agentpool_decorator.py |
Tests zone mutation and serialization. |
src/aks-preview/azext_aks_preview/custom.py |
Adds zones to the update command. |
src/aks-preview/azext_aks_preview/agentpool_decorator.py |
Applies zone updates with core compatibility handling. |
src/aks-preview/azext_aks_preview/_params.py |
Registers --zones and -z. |
src/aks-preview/azext_aks_preview/_help.py |
Adds help documentation and an example. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
CI follow-up: build 341763 exposed two repository-baseline cassette API-version drifts unrelated to |
|
AKS |
🤖 PR Validation —⚠️ Review suggested
Related command
az aks nodepool update --zones autoDescription
Adds preview
--zones/-zsupport toaks-previewnode pool update and bumps the extension to22.0.0b4. A regional VMSS node pool can migrate to automatic zone placement with--zones auto; omitting the option preserves the current setting. The option composes with the existing--node-vm-sizeresize flow.The implementation remains compatible with both current CLI core and a future core that handles zones in its default decorator flow, without applying the update twice. Other availability-zone changes remain subject to service restrictions.
Companion official CLI PR: Azure/azure-cli#33976
Validation
az aks alert-config: Add commands to manage alert configurations #10155 / ADO build 341562.azdev linter aks-previewandgit diff --checkpassed.azdev style aks-previewwas run; the repository-wide command exits on existing missing optional dependencies and vendored SDK/build-tree lint findings unrelated to this diff.aks-preview 22.0.0b4was loaded and help exposes--zones -z [Preview].extpool:availabilityZones: nullto["auto"], final stateSucceeded.combopool:--zones auto --node-vm-size Standard_D4s_v3, final stateSucceeded; the profile reported["auto"]andStandard_D4s_v3, and the replacement VMSS instance was placed in zone 1.General Guidelines
azdev style aks-previewlocally (baseline issues noted above).python scripts/ci/test_index.py -qlocally.