Skip to content

Conversation

@mainred
Copy link
Member

@mainred mainred commented Aug 15, 2025


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

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
  • My extension version conforms to the Extension version schema

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Copilot AI review requested due to automatic review settings August 15, 2025 08:02
@azure-client-tools-bot-prd
Copy link

azure-client-tools-bot-prd bot commented Aug 15, 2025

⚠️Azure CLI Extensions Breaking Change Test
⚠️aks-preview
rule cmd_name rule_message suggest_message
⚠️ 1001 - CmdAdd aks agent cmd aks agent added

@yonzhan
Copy link
Collaborator

yonzhan commented Aug 15, 2025

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

@github-actions
Copy link

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>

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enables the AI-powered debugging tool az aks agent for Azure Kubernetes Service by uncommenting previously disabled code and updating dependencies.

Key changes:

  • Enables the az aks agent command for interactive AI-powered cluster troubleshooting
  • Updates the holmesgpt dependency to version 0.12.5
  • Refines error messaging format in the agent prompt system

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
setup.py Bumps version to 18.0.0b27 and updates holmesgpt dependency to 0.12.5
commands.py Uncomments the az aks agent command registration to enable the feature
_params.py Uncomments parameter definitions for the agent command and re-enables imports
_help.py Uncomments comprehensive help documentation for the agent command
agent/prompt.py Updates error message format for cluster context failures
HISTORY.rst Adds changelog entry for the new agent feature

@github-actions
Copy link

github-actions bot commented Aug 15, 2025

@github-actions github-actions bot added the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 20, 2025
@github-actions github-actions bot removed the release-version-block Updates do not qualify release version rules. NOTE: please do not edit it manually. label Aug 21, 2025
@mainred
Copy link
Member Author

mainred commented Aug 24, 2025

@FumingZhang @yanzhudd

@yanzhudd
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@yanzhudd
Copy link
Contributor

could you please add some tests for the new command?

@mainred
Copy link
Member Author

mainred commented Aug 25, 2025

@yanzhudd I have added UT, please take another look, thanks.

@yanzhudd
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@yanzhudd
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

Copy link
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

@yanzhudd yanzhudd merged commit 455e0ce into Azure:main Aug 26, 2025
25 checks passed
@azclibot
Copy link
Collaborator

[Release] Update index.json for extension [ aks-preview-18.0.0b28 ] : https://dev.azure.com/msazure/One/_build/results?buildId=135035918&view=results

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

AKS Auto-Assign Auto assign by bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants