Skip to content
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

[Storage] az storage share-rm: Support create/show/delete snapshots #17449

Merged
merged 4 commits into from
Apr 8, 2021

Conversation

evelyn-ys
Copy link
Member

Description

File share management plane operations support snapshot CRD. (List operation is not covered in this PR because list including both deleted and snapshots is not supported which is a known issue will be resolved in next api version.)

Related swagger: Azure/azure-rest-api-specs#11372

Testing Guide

Create: az storage share-rm snapshot
Show: az storage share-rm show --snapshot
Delete: az storage share-rm delete --snapshot


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

@evelyn-ys evelyn-ys self-assigned this Mar 25, 2021
@evelyn-ys evelyn-ys changed the title [Storage] az storage share-rm: Support create/show/delete snapshot [Storage] az storage share-rm: Support create/show/delete snapshots Mar 25, 2021
@evelyn-ys
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@yonzhan yonzhan modified the milestones: S184, S185 Mar 25, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented Mar 25, 2021

Storage

g.generic_update_command('update', setter_name='update', setter_arg_name='file_share',
custom_func_name='update_share_rm')
g.custom_command('stats', 'get_stats', transform=lambda x: getattr(x, 'share_usage_bytes'))
g.custom_command('restore', 'restore_share_rm')
g.custom_command('snapshot', 'snapshot_share_rm', min_api='2020-08-01-preview',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you add preview tag for new commands?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added for new command and new parameter.

@evelyn-ys evelyn-ys requested a review from Juliehzl March 31, 2021 02:08
@evelyn-ys evelyn-ys merged commit 4b7980c into Azure:dev Apr 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants