[AKS] Add --enable-hosted-system, --system-node-subnet-id and --node-subnet-id to az aks update - #33928
Closed
reneeli123 wants to merge 5 commits into
Closed
[AKS] Add --enable-hosted-system, --system-node-subnet-id and --node-subnet-id to az aks update#33928reneeli123 wants to merge 5 commits into
--enable-hosted-system, --system-node-subnet-id and --node-subnet-id to az aks update#33928reneeli123 wants to merge 5 commits into
Conversation
|
Hi reneeli123, |
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
microsoft-github-policy-service
Bot
requested review from
FumingZhang,
elvazhu521,
Julie Zhu (yanzhudd) and
Yong Zhang (yonzhan)
August 19, 2026 02:17
microsoft-github-policy-service
Bot
added
Auto-Assign
reneeli123
marked this pull request as ready for review
August 19, 2026 02:19
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds support for converting an existing AKS Automatic cluster to a Managed System Pool (hosted system / “HOBO”) cluster via az aks update, bringing the update path closer to feature parity with az aks create for hosted-system conversion and optional BYO VNet subnet inputs.
Changes:
- Adds
--enable-hosted-system,--system-node-subnet-id, and--node-subnet-idtoaz aks update(params + command signature + help). - Extends hosted-system enablement + subnet validation to run in UPDATE mode, and adds
update_hosted_system_profile()to sethostedSystemProfileduring update. - Adds unit tests for the new update behavior and linter exclusions for the new parameters.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_managed_cluster_decorator.py | Adds unit tests covering hosted-system profile updates and validation on aks update. |
| src/azure-cli/azure/cli/command_modules/acs/managed_cluster_decorator.py | Enables hosted-system validation in UPDATE mode and adds update_hosted_system_profile() to populate hostedSystemProfile during update. |
| src/azure-cli/azure/cli/command_modules/acs/linter_exclusions.yml | Excludes missing parameter test coverage lints for the new aks update parameters. |
| src/azure-cli/azure/cli/command_modules/acs/custom.py | Adds new arguments to the aks_update command function signature. |
| src/azure-cli/azure/cli/command_modules/acs/_params.py | Wires new CLI arguments (with validators / store_true) into aks update. |
| src/azure-cli/azure/cli/command_modules/acs/_help.py | Documents the new aks update flags and provides examples for hosted-system conversion. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…bnet-id to az aks update
reneeli123
force-pushed
the
reneel/hobo-migration
branch
from
August 19, 2026 03:09
2dfe2f4 to
1e040ed
Compare
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Aditya Pujara (a0x1ab)
added
the
Azure Client Tools Agent Requested
Member
|
please fix CI error first reneeli123 |
Contributor
Live test results —
|
Contributor
Author
|
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
Author
elvazhu521 Transient issue, the same step passed previously https://dev.azure.com/azclitools/public/_build/results?buildId=339434&view=logs&s=96ac2280-8cb4-5df5-99de-dd2da759617d&j=e2f732ac-2c19-5c7d-b3eb-bbf48e0f6d6e Just retriggered CI. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Contributor
Live test results —
|
![azure-client-tools-agent[bot]](https://avatars.githubusercontent.com/in/4205534?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Automated CI Summary
The current head commit (53b16c5) has a failing check:
- Azure.azure-cli (Build) —
failure: Build #20260819.22 failed
Please review the build log for the failure and push a fix. CI will re-run automatically on the next push.
azure-client-tools-agent
Bot
added
azure-client-tools-agent
Contributor
|
🔔 Routing this PR to @Azure/act-observability-squad. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
elvazhu521
approved these changes
Aug 20, 2026
Member
|
reneeli123 there's a merge conflict in the PR |
Member
|
Copilot resolve the merge conflicts in this pull request |
1 similar comment
Member
|
Copilot resolve the merge conflicts in this pull request |
3 tasks
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Contributor
Author
|
Discussed with PM we decided to add this as a preview feature first before go directly to GA, so I close the PR first. Thanks FumingZhang elvazhu521 ! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 PR Validation — ❌ Action needed
❌AzureCLI-FullTest
Related command
plain non-HOBO to HOBO conversion, keeps current AKS-managed networking
az aks update -g $RG -n $CLUSTER --enable-hosted-system
non-HOBO BYO VNet to HOBO conversion with system node pool subnet supplied
az aks update -g $RG -n $CLUSTER --enable-hosted-system --system-node-subnet-id $SYS
non-HOBO BYO VNet to HOBO conversion with system + user node pools subnet supplied
az aks update -g $RG -n $CLUSTER --enable-hosted-system --system-node-subnet-id $SYS --node-subnet-id $NODE
Description
Adds
--enable-hosted-system,--system-node-subnet-idand--node-subnet-idtoaz aks update, so an existing Automatic cluster can be converted to a Managed System Pool (HOBO) cluster in place. These flags already existed onaz aks create; this brings the update path to parity.Ports Azure/azure-cli-extensions#10199 and its follow-up Azure/azure-cli-extensions#10227.
get_enable_hosted_system()andvalidate_byo_hosted_system_subnets()now run inUPDATEmode; both previously early-returned outsideCREATE.--enable-hosted-system— unlike create, the subnets never imply it.--apiserver-subnet-idis not required on update;--node-subnet-idrequires--system-node-subnet-id. Omitted subnets leave the cluster's current networking untouched.update_hosted_system_profile()setshostedSystemProfile.{enabled,systemNodeSubnetId,nodeSubnetId}. Create-path behaviour is unchanged.Testing Guide
History Notes
[AKS]
az aks update: Add--enable-hosted-system,--system-node-subnet-idand--node-subnet-idto support converting an existing non-HOBO Automatic cluster to a Managed System Pool (HOBO) cluster, optionally with BYO VNetThis checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.