Skip to content

[AKS] az aks create/update: Add new parameter --enable-ai-toolchain-operator to enable Kaito addon#31485

Merged
zhoxing-ms merged 4 commits intoAzure:devfrom
helayoty:helayoty/kaito
Jul 22, 2025
Merged

[AKS] az aks create/update: Add new parameter --enable-ai-toolchain-operator to enable Kaito addon#31485
zhoxing-ms merged 4 commits intoAzure:devfrom
helayoty:helayoty/kaito

Conversation

@helayoty
Copy link
Copy Markdown
Contributor

@helayoty helayoty commented May 16, 2025

Related command
az aks create
az aks update

Description
Add --enable-ai-toolchain-operator and --disable-ai-toolchain-operator to az aks cli.

Testing Guide

  • az aks create <cluster_name> -g <resource_group_name> --enable-ai-toolchain-operator
  • az aks update <cluster_name> -g <resource_group_name> --enable-ai-toolchain-operator
  • az aks update <cluster_name> -g <resource_group_name> --disable-ai-toolchain-operator

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature


This checklist is used to make sure that common guidelines for a pull request are followed.

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented May 16, 2025

️✔️AzureCLI-FullTest
️✔️acr
️✔️latest
️✔️3.12
️✔️3.9
️✔️acs
️✔️latest
️✔️3.12
️✔️3.9
️✔️advisor
️✔️latest
️✔️3.12
️✔️3.9
️✔️ams
️✔️latest
️✔️3.12
️✔️3.9
️✔️apim
️✔️latest
️✔️3.12
️✔️3.9
️✔️appconfig
️✔️latest
️✔️3.12
️✔️3.9
️✔️appservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️aro
️✔️latest
️✔️3.12
️✔️3.9
️✔️backup
️✔️latest
️✔️3.12
️✔️3.9
️✔️batch
️✔️latest
️✔️3.12
️✔️3.9
️✔️batchai
️✔️latest
️✔️3.12
️✔️3.9
️✔️billing
️✔️latest
️✔️3.12
️✔️3.9
️✔️botservice
️✔️latest
️✔️3.12
️✔️3.9
️✔️cdn
️✔️latest
️✔️3.12
️✔️3.9
️✔️cloud
️✔️latest
️✔️3.12
️✔️3.9
️✔️cognitiveservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️compute_recommender
️✔️latest
️✔️3.12
️✔️3.9
️✔️computefleet
️✔️latest
️✔️3.12
️✔️3.9
️✔️config
️✔️latest
️✔️3.12
️✔️3.9
️✔️configure
️✔️latest
️✔️3.12
️✔️3.9
️✔️consumption
️✔️latest
️✔️3.12
️✔️3.9
️✔️container
️✔️latest
️✔️3.12
️✔️3.9
️✔️containerapp
️✔️latest
️✔️3.12
️✔️3.9
️✔️core
️✔️latest
️✔️3.12
️✔️3.9
️✔️cosmosdb
️✔️latest
️✔️3.12
️✔️3.9
️✔️databoxedge
️✔️latest
️✔️3.12
️✔️3.9
️✔️dls
️✔️latest
️✔️3.12
️✔️3.9
️✔️dms
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventgrid
️✔️latest
️✔️3.12
️✔️3.9
️✔️eventhubs
️✔️latest
️✔️3.12
️✔️3.9
️✔️feedback
️✔️latest
️✔️3.12
️✔️3.9
️✔️find
️✔️latest
️✔️3.12
️✔️3.9
️✔️hdinsight
️✔️latest
️✔️3.12
️✔️3.9
️✔️identity
️✔️latest
️✔️3.12
️✔️3.9
️✔️iot
️✔️latest
️✔️3.12
️✔️3.9
️✔️keyvault
️✔️latest
️✔️3.12
️✔️3.9
️✔️lab
️✔️latest
️✔️3.12
️✔️3.9
️✔️managedservices
️✔️latest
️✔️3.12
️✔️3.9
️✔️maps
️✔️latest
️✔️3.12
️✔️3.9
️✔️marketplaceordering
️✔️latest
️✔️3.12
️✔️3.9
️✔️monitor
️✔️latest
️✔️3.12
️✔️3.9
️✔️mysql
️✔️latest
️✔️3.12
️✔️3.9
️✔️netappfiles
️✔️latest
️✔️3.12
️✔️3.9
️✔️network
️✔️latest
️✔️3.12
️✔️3.9
️✔️policyinsights
️✔️latest
️✔️3.12
️✔️3.9
️✔️privatedns
️✔️latest
️✔️3.12
️✔️3.9
️✔️profile
️✔️latest
️✔️3.12
️✔️3.9
️✔️rdbms
️✔️latest
️✔️3.12
️✔️3.9
️✔️redis
️✔️latest
️✔️3.12
️✔️3.9
️✔️relay
️✔️latest
️✔️3.12
️✔️3.9
️✔️resource
️✔️latest
️✔️3.12
️✔️3.9
️✔️role
️✔️latest
️✔️3.12
️✔️3.9
️✔️search
️✔️latest
️✔️3.12
️✔️3.9
️✔️security
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicebus
️✔️latest
️✔️3.12
️✔️3.9
️✔️serviceconnector
️✔️latest
️✔️3.12
️✔️3.9
️✔️servicefabric
️✔️latest
️✔️3.12
️✔️3.9
️✔️signalr
️✔️latest
️✔️3.12
️✔️3.9
️✔️sql
️✔️latest
️✔️3.12
️✔️3.9
️✔️sqlvm
️✔️latest
️✔️3.12
️✔️3.9
️✔️storage
️✔️latest
️✔️3.12
️✔️3.9
️✔️synapse
️✔️latest
️✔️3.12
️✔️3.9
️✔️telemetry
️✔️latest
️✔️3.12
️✔️3.9
️✔️util
️✔️latest
️✔️3.12
️✔️3.9
️✔️vm
️✔️latest
️✔️3.12
️✔️3.9

@azure-client-tools-bot-prd
Copy link
Copy Markdown

azure-client-tools-bot-prd bot commented May 16, 2025

⚠️AzureCLI-BreakingChangeTest
⚠️acs
rule cmd_name rule_message suggest_message
⚠️ 1006 - ParaAdd aks create cmd aks create added parameter enable_ai_toolchain_operator
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter disable_ai_toolchain_operator
⚠️ 1006 - ParaAdd aks update cmd aks update added parameter enable_ai_toolchain_operator

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented May 16, 2025

Thank you for your contribution! We will review the pull request and get back to you soon.

@github-actions
Copy link
Copy Markdown

The git hooks are available for azure-cli and azure-cli-extensions repos. They could help you run required checks before creating the PR.

Please sync the latest code with latest dev branch (for azure-cli) or main branch (for azure-cli-extensions).
After that please run the following commands to enable git hooks:

pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>

@FumingZhang
Copy link
Copy Markdown
Member

Depends on SDK generated based on 2025-05-01

@FumingZhang
Copy link
Copy Markdown
Member

updated milestone to https://github.com/Azure/azure-cli/milestone/157

Copy link
Copy Markdown
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Queued live test to validate the change.

  • test_aks_create_with_enable_ai_toolchain_operator
  • test_aks_update_with_enable_ai_toolchain_operator

@yanzhudd yanzhudd changed the title [AKS] az aks create/update: Add new parameter --enable-ai-toolchain-operator to enable Kaito addon [AKS] az aks create/update: Add new parameter --enable-ai-toolchain-operator to enable Kaito addon Jun 23, 2025
@yanzhudd
Copy link
Copy Markdown
Contributor

Hi @FumingZhang
is this PR ready to review and merge?

@yanzhudd
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@FumingZhang
Copy link
Copy Markdown
Member

Queued live test to validate the change.

  • test_aks_create_with_enable_ai_toolchain_operator
  • test_aks_update_with_enable_ai_toolchain_operator

Re-queued live test after rebasing the branch from dev to include the SDK version update. https://github.com/FumingZhang/azure-cli/tree/fuming/helayoty/kaito

@FumingZhang
Copy link
Copy Markdown
Member

Queued live test to validate the change.

  • test_aks_create_with_enable_ai_toolchain_operator
  • test_aks_update_with_enable_ai_toolchain_operator

Re-queued live test after rebasing the branch from dev to include the SDK version update. https://github.com/FumingZhang/azure-cli/tree/fuming/helayoty/kaito

failed with error

E AttributeError: 'ManagedCluster' object has no attribute 'ai_toolchain_operator_profile'

https://dev.azure.com/msazure/CloudNativeCompute/_build/results?buildId=128092330&view=logs&j=b162b355-d59d-5864-ce0f-0a70f12dd28b&t=dc59ccd1-231f-538b-777f-33a592c7ca57&l=2702

@yanzhudd
Copy link
Copy Markdown
Contributor

please fix the testing issues.

@zhoxing-ms
Copy link
Copy Markdown
Contributor

Please note that Azure CLI will have a code freeze on 06/24/2025 07:00 UTC for the upcoming release. If you want to catch this release train, please resolve the CI issue ASAP, otherwise it has to be postponed to next sprint (08-05).

@yanzhudd
Copy link
Copy Markdown
Contributor

/azp run

@FumingZhang
Copy link
Copy Markdown
Member

Depends on SDK generated based on 2025-05-01

the sdk is not ready yet, moving to next milestone
https://github.com/Azure/azure-cli/milestone/158

@helayoty
Copy link
Copy Markdown
Contributor Author

Queued live test to validate the change after rebasing.

  • test_aks_create_with_enable_ai_toolchain_operator
  • test_aks_update_with_enable_ai_toolchain_operator

Result: Passed

@azure-pipelines
Copy link
Copy Markdown

Commenter does not have sufficient privileges for PR 31485 in repo Azure/azure-cli

@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Jul 17, 2025

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@FumingZhang
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

Copy link
Copy Markdown
Member

@FumingZhang FumingZhang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Comment thread src/azure-cli/azure/cli/command_modules/acs/_params.py Outdated
Comment thread src/azure-cli/azure/cli/command_modules/acs/_params.py Outdated
@helayoty
Copy link
Copy Markdown
Contributor Author

Queued live test to validate the change after addressing the feedback rebasing.

@helayoty helayoty requested a review from FumingZhang July 19, 2025 01:01
@FumingZhang
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@FumingZhang
Copy link
Copy Markdown
Member

Newly added test cases failed in replay mode
https://dev.azure.com/azclitools/public/_build/results?buildId=261975&view=logs&j=c4905e22-e505-5bfe-aa0a-49d3f9325fa2&t=cf247c3b-aad8-52b9-12ca-ceb3e5942fcf&l=4222

FAILED src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_aks_commands.py::AzureKubernetesServiceScenarioTest::test_aks_create_with_enable_ai_toolchain_operator
FAILED src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_aks_commands.py::AzureKubernetesServiceScenarioTest::test_aks_update_with_enable_ai_toolchain_operator

Newly queued live test will help generate the correct recording files you'll need. Please download them from pipeline artifacts and commit them accordingly.

helayoty added 4 commits July 21, 2025 17:11
Signed-off-by: Heba <31887807+helayoty@users.noreply.github.com>
Signed-off-by: Heba <31887807+helayoty@users.noreply.github.com>
Signed-off-by: Heba <31887807+helayoty@users.noreply.github.com>
Signed-off-by: Heba <31887807+helayoty@users.noreply.github.com>
@helayoty
Copy link
Copy Markdown
Contributor Author

helayoty commented Jul 21, 2025

Newly added test cases failed in replay mode https://dev.azure.com/azclitools/public/_build/results?buildId=261975&view=logs&j=c4905e22-e505-5bfe-aa0a-49d3f9325fa2&t=cf247c3b-aad8-52b9-12ca-ceb3e5942fcf&l=4222

FAILED src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_aks_commands.py::AzureKubernetesServiceScenarioTest::test_aks_create_with_enable_ai_toolchain_operator
FAILED src/azure-cli/azure/cli/command_modules/acs/tests/latest/test_aks_commands.py::AzureKubernetesServiceScenarioTest::test_aks_update_with_enable_ai_toolchain_operator

Newly queued live test will help generate the correct recording files you'll need. Please download them from pipeline artifacts and commit them accordingly.

Addressed and replaced. However I didn't see the failure in the latest running pipeline you mentioned.

Latest live test passed.

@FumingZhang
Copy link
Copy Markdown
Member

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 3 pipeline(s).

@zhoxing-ms zhoxing-ms merged commit db8472d into Azure:dev Jul 22, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AKS az aks/acs/openshift Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants