[Compute] az vm availability-set update: Add new parameter --enable-all-instance-down to support setting scheduled events profile#32285
Conversation
️✔️AzureCLI-FullTest
|
|
Hi @Jing-song, |
|
| rule | cmd_name | rule_message | suggest_message |
|---|---|---|---|
| vm availability-set convert | cmd vm availability-set convert added parameter enable_all_instance_down |
||
| vm availability-set convert | cmd vm availability-set convert added parameter scheduled_events_api_version |
||
| vm availability-set update | cmd vm availability-set update added parameter enable_all_instance_down |
||
| vm availability-set update | cmd vm availability-set update added parameter scheduled_events_api_version |
|
Thank you for your contribution! We will review the pull request and get back to you soon. |
|
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
Adds support for configuring Availability Set Scheduled Events policy via new parameters and updates API version references to 2025-04-01.
- Introduces parameters to auto-approve when all instances are down and to set scheduled events API version.
- Expands read/list schemas with new scheduled events fields and system data, plus version bump across CRUD commands.
- Adds tests and recordings for new parameters, though negative-only coverage for scheduled events API version.
Reviewed Changes
Copilot reviewed 7 out of 9 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| test_vm_commands.py | Adds test for new parameters (--enable-all-instance-down and --scheduled-events-api-version) including negative scenario. |
| test_vm_avset_scheduled_events_policy_parameters.yaml | Recording for new test validating allInstancesDown and failed scheduledEventsApiVersion request. |
| test_vm_availset_scheduled_events_policy.yaml | Updated recordings reflecting new API version and additional read-only fields. |
| test_ppg_update.yaml | Updates API version used when fetching/updating availability sets in PPG test context. |
| _update.py | Implements new arguments, maps them into scheduledEventsPolicy, adds schema fields (allInstancesDown, scheduledEventsApiVersion, migration info, system data) and bumps API version. |
| _list.py | Updates list command to new API version and augments response schema with added scheduled events and system data fields. |
| _delete.py | Bumps delete command API version to 2025-04-01. |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Related command
az vm availability-set updateDescription
ScheduledEventsProfile #32056
aaz: Azure/aaz#878
Testing Guide
History Notes
[Compute]
az vm availability-set update: Add new parameter--enable-all-instance-downto support setting scheduled events profile[Compute]
az vm availability-set update: Add new parameter--scheduled-events-api-versionto support setting scheduled events profileThis checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.