Skip to content

Conversation

@xfz11
Copy link
Contributor

@xfz11 xfz11 commented Apr 2, 2021

Description

New command vm install-patches
Resolve #16790

Example
az vm install-patches -g MyResourceGroup -n MyVm --maximum-duration PT4H --reboot-setting IfRequired --classifications-to-include Critical Security --exclude-kbs-requiring-reboot true

Testing Guide

        self.cmd(
            'vm create -g {rg} -n vm --image Win2019Datacenter --enable-hotpatching true --admin-username azureuser --admin-password testPassword0 --nsg-rule NONE')
        self.cmd('vm install-patches -g {rg} -n vm --maximum-duration PT4H --reboot-setting IfRequired '
                 '--classifications-to-include Critical Security --exclude-kbs-requiring-reboot true', checks=[
            self.check('status', 'Succeeded')
        ])

History Notes


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

@xfz11 xfz11 requested review from houk-ms and qwordy April 2, 2021 05:13
@qwordy
Copy link
Member

qwordy commented Apr 2, 2021

I suggest develop features in your own forked repository.

@yungezz yungezz added the Compute az vm/vmss/image/disk/snapshot label Apr 6, 2021
@xfz11 xfz11 marked this pull request as ready for review April 6, 2021 09:34
@xfz11 xfz11 requested review from Juliehzl and yungezz as code owners April 6, 2021 09:34
@xfz11
Copy link
Contributor Author

xfz11 commented Apr 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@xfz11
Copy link
Contributor Author

xfz11 commented Apr 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@xfz11
Copy link
Contributor Author

xfz11 commented Apr 8, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@xfz11
Copy link
Contributor Author

xfz11 commented Apr 9, 2021

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

update install-patches test case

update as comments

lint

update test recording

add test case

Update src/azure-cli-core/azure/cli/core/profiles/_shared.py

Co-authored-by: Feiyue Yu <[email protected]>

Update src/azure-cli/azure/cli/command_modules/vm/_params.py

Co-authored-by: Feiyue Yu <[email protected]>

Update src/azure-cli/azure/cli/command_modules/vm/custom.py

Co-authored-by: Feiyue Yu <[email protected]>

update as comments

Update src/azure-cli/azure/cli/command_modules/vm/custom.py

Co-authored-by: Feiyue Yu <[email protected]>

update as comments

update

add client in the arguments list of install_vm_patches function

remove short name of arguments

disable option_length_too_long lint

remove unnecessary comma

refactor arguments

lint

lint option_length_too_long

add help info
@xfz11 xfz11 merged commit 3b813cb into dev Apr 9, 2021
@jiasli jiasli deleted the issue16790 branch August 5, 2021 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Compute az vm/vmss/image/disk/snapshot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

New CLI command for install patches API

3 participants