[AKS] az aks nodepool add/update: Add --enable-managed-dranet - #34003
Open
Evan Baker (rbtr) wants to merge 2 commits into
Open
[AKS] az aks nodepool add/update: Add --enable-managed-dranet#34003Evan Baker (rbtr) wants to merge 2 commits into
az aks nodepool add/update: Add --enable-managed-dranet#34003Evan Baker (rbtr) wants to merge 2 commits into
Conversation
|
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
elvazhu521,
Julie Zhu (yanzhudd) and
Yong Zhang (yonzhan)
August 27, 2026 17:55
Contributor
|
Thank you for your contribution Evan Baker (@rbtr)! We will review the pull request and get back to you soon. |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds first-class support in core azure-cli for enabling Managed DRANET on AKS node pools via az aks nodepool add and az aks nodepool update, including plumbing through parameters/decorators and updating the ContainerService SDK dependency to a version that contains the necessary models.
Changes:
- Add
--enable-managed-dranetto nodepool add/update args, help, and the agentpool decorator logic to setnetworkProfile.dranet.mode=Managed. - Upgrade
azure-mgmt-containerservicefrom41.5.0to41.6.0across CLI packaging/requirements. - Add unit test coverage for the new context getter and decorator network-profile behavior.
Show a summary per file
| File | Description |
|---|---|
| src/azure-cli/setup.py | Bumps azure-mgmt-containerservice dependency range to ~= 41.6.0. |
| src/azure-cli/requirements.py3.windows.txt | Pins azure-mgmt-containerservice to 41.6.0 on Windows. |
| src/azure-cli/requirements.py3.Linux.txt | Pins azure-mgmt-containerservice to 41.6.0 on Linux. |
| src/azure-cli/requirements.py3.Darwin.txt | Pins azure-mgmt-containerservice to 41.6.0 on macOS. |
| src/azure-cli/HISTORY.rst | Adds release note entry for the new nodepool flag. |
| src/azure-cli/azure/cli/command_modules/acs/custom.py | Wires enable_managed_dranet through add/update command handlers via raw_parameters = locals(). |
| src/azure-cli/azure/cli/command_modules/acs/agentpool_decorator.py | Implements context getter and applies Managed DRANET into AgentPoolNetworkProfile. |
| src/azure-cli/azure/cli/command_modules/acs/_params.py | Adds enable_managed_dranet as a store_true argument for nodepool add/update. |
| src/azure-cli/azure/cli/command_modules/acs/_help.py | Documents --enable-managed-dranet for nodepool add/update. |
| src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_agentpool_decorator.py | Adds unit tests for DRANET getter and network profile setup/update behavior. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 10/10 changed files
- Comments generated: 2
- Review effort level: Lite
Comment on lines
+1789
to
+1791
| def get_enable_managed_dranet(self) -> bool: | ||
| """Obtain the value of enable_managed_dranet.""" | ||
| return self.raw_param.get("enable_managed_dranet") |
|
|
||
| **AKS** | ||
|
|
||
| * `az aks nodepool add/update`: Add `--enable-managed-dranet` to enable Managed DRANET on a node pool. |
Member
|
/azp run |
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
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-BreakingChangeTest
Please submit your Breaking Change Pre-announcement ASAP if you haven't already. Please note:
❌AzureCLI-FullTest
Related command
az aks nodepool addaz aks nodepool updateDescription
Adds
--enable-managed-dranetto AKS node pool create and update commands. When specified, the command sendsnetworkProfile.dranet.mode=Managed; when omitted during update, the existing DRANET profile is preserved.The change upgrades
azure-mgmt-containerservicefrom41.5.0to41.6.0, which uses the stable2026-06-01API and contains the released DRANET AgentPool models. Preview support can ship earlier through Azure/azure-cli-extensions#10266, while this PR follows the core CLI release train.Testing Guide
Validated with:
azdev style acsazdev linter acs --min-severity mediumazdev test azure-cli-acs -a -k managed_dranetHistory Notes
[AKS]
az aks nodepool add/update: Add--enable-managed-dranetto enable Managed DRANET on a node pool.