azure-cli icon indicating copy to clipboard operation
azure-cli copied to clipboard

[App Service] `az webapp config backup delete`: Command to delete a backup of the webapp

Open vkarasala opened this issue 2 years ago • 1 comments

Related command

webapp config backup delete is the new command introduced with this PR

Description

The new command enables customers to delete a backup by id

Testing Guide

List the web app backups by using the command az webapp config backup list Sample commands : For the app : az webapp config backup list -g test_group --webapp-name test-container-appa For the app's slot : az webapp config backup list -g test_group --webapp-name test-container-appa -s dev

Pick an an backup from the list and delete the backup using the command az webapp config backup delete For the app : az webapp config backup delete -g test_group --webapp-name test-container-appa --backup-id 89215 For the app's slot : az webapp config backup delete -g test_group --webapp-name test-container-appa --backup-id 89216 --slot dev

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 feature


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

vkarasala avatar Jan 19 '23 07:01 vkarasala

App Service

yonzhan avatar Jan 19 '23 09:01 yonzhan

@vkarasala Can you please check if you can add the tests ?

https://github.com/Azure/azure-cli/tree/dev/src/azure-cli/azure/cli/command_modules/appservice/tests/latest

sulabh-msft avatar Jan 30 '23 07:01 sulabh-msft

@vkarasala Please note that since we will launch the release of this sprint the day after tomorrow. If you cannot address all the comments tomorrow, we will have to postpone the release of this PR to the next sprint (03-07)

zhoxing-ms avatar Feb 01 '23 10:02 zhoxing-ms

@vkarasala Please note that since we will launch the release of this sprint the day after tomorrow. If you cannot address all the comments tomorrow, we will have to postpone the release of this PR to the next sprint (03-07)

@zhoxing-ms Thanks for the review, resolved your comments.

vkarasala avatar Feb 01 '23 11:02 vkarasala

Can you please check if you can add the tests ? https://github.com/Azure/azure-cli/tree/dev/src/azure-cli/azure/cli/command_modules/appservice/tests/latest

@vkarasala Could you please address this comment?

zhoxing-ms avatar Feb 02 '23 02:02 zhoxing-ms

@vkarasala Could you resolve the CI issues?

zhoxing-ms avatar Feb 02 '23 02:02 zhoxing-ms