-
Notifications
You must be signed in to change notification settings - Fork 3.3k
{Cognitive Services} Add agent status command #32691
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
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.
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 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.