[Compute] az vm run-command invoke: Fix --no-wait not working#32979
[Compute] az vm run-command invoke: Fix --no-wait not working#32979
az vm run-command invoke: Fix --no-wait not working#32979Conversation
️✔️AzureCLI-FullTest
|
️✔️AzureCLI-BreakingChangeTest
|
|
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
Fixes az vm run-command invoke so --no-wait is correctly propagated to the underlying AAZ Invoke command, avoiding failures and enabling true async behavior.
Changes:
- Add
no_waitparameter plumbing invm_run_command_invoke/run_command_invokeand forward it into AAZ command args. - Update parts of a VM custom image live test (VM size changes and removal of a VMSS validation block).
Reviewed changes
Copilot reviewed 2 out of 3 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
src/azure-cli/azure/cli/command_modules/vm/custom.py |
Wires no_wait through the custom wrapper into the AAZ run-command invoke implementation. |
src/azure-cli/azure/cli/command_modules/vm/tests/latest/test_vm_commands.py |
Adjusts VM sizes used in a custom image test and removes a VMSS creation/assertion segment. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
You can also share your feedback on Copilot code review. Take the survey.
| self.check('storageProfile.imageReference.resourceGroup', '{rg}'), | ||
| self.check('storageProfile.osDisk.createOption', 'FromImage') | ||
| ]) | ||
|
|
| self.cmd('vm create -g {rg} -n {vm1} --image Debian:debian-10:10:latest --use-unmanaged-disk --admin-username sdk-test-admin ' | ||
| '--admin-password testPassword0 --subnet {subnet} --vnet-name {vnet} --size Standard_B2ms --nsg-rule NONE') | ||
| '--admin-password testPassword0 --subnet {subnet} --vnet-name {vnet} --size Standard_D2s_v3 --nsg-rule NONE') |
Related command
az vm run-command invokeDescription
Fix IcM: 21000000922941
Testing Guide
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 featureThis 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.