{Cognitive Services} Add agent status command#32691
Conversation
️✔️AzureCLI-FullTest
|
|
Hi @zzn2, |
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| cognitiveservices agent status | cmd cognitiveservices agent status added |
|
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). pip install azdev --upgrade
azdev setup -c <your azure-cli repo path> -r <your azure-cli-extensions repo path>
|
There was a problem hiding this comment.
Pull request overview
This PR adds a new agent status command to retrieve the deployment status of a hosted agent from the AI Projects data-plane API.
Changes:
- Added
agent_statusfunction to retrieve hosted agent deployment status - Registered new
statuscommand in the agent command group - Added parameter definitions including
--only-show-statusflag to filter output - Added comprehensive help documentation with usage examples
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| src/azure-cli/azure/cli/command_modules/cognitiveservices/custom.py | Implements agent_status function that calls _get_agent_container_status and optionally filters to return only the status field |
| src/azure-cli/azure/cli/command_modules/cognitiveservices/commands.py | Registers the new status command using custom_show_command |
| src/azure-cli/azure/cli/command_modules/cognitiveservices/_params.py | Defines required agent_version parameter and optional --only-show-status flag |
| src/azure-cli/azure/cli/command_modules/cognitiveservices/_help.py | Provides command documentation with short/long summaries and usage examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
de5d489 to
21701a0
Compare
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
it would be great to add some test coverage. |
|
@necusjz Thanks for the comments. I added unit test, could you help review? |
|
/azp run |
|
Azure Pipelines successfully started running 3 pipeline(s). |
|
@necusjz It seems the checks are passed. It it good for us to merge the PR? |
Adds to retrieve the hosted agent deployment status from the AI Projects data-plane API.\n\nNotes:\n- Includes to return only.