Skip to content

{Storage} Add help example#17550

Merged
Juliehzl merged 4 commits intoAzure:devfrom
zhangyan133:zy-storage-2021-04-02
Apr 26, 2021
Merged

{Storage} Add help example#17550
Juliehzl merged 4 commits intoAzure:devfrom
zhangyan133:zy-storage-2021-04-02

Conversation

@zhangyan133
Copy link
Copy Markdown
Contributor

@zhangyan133 zhangyan133 commented Apr 2, 2021

Description

Fix #16743

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 feature.


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

@yonzhan yonzhan added this to the S185 milestone Apr 2, 2021
@yonzhan
Copy link
Copy Markdown
Collaborator

yonzhan commented Apr 2, 2021

storage example

@zhangyan133 zhangyan133 changed the title storage account management-policy update add example {Storage} Add help example Apr 6, 2021
@zhangyan133
Copy link
Copy Markdown
Contributor Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

examples:
- name: Update the data policy rules associated with the specified storage account.
text: |
az storage account management-policy update --account-name myaccount --resource-group myresourcegroup
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

where is your policy to update?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

right, I have modified it.

Yan Zhang (WICRESOFT NORTH AMERICA LTD) added 2 commits April 6, 2021 14:17
examples:
- name: Update the data policy rules associated with the specified storage account.
text: |
az storage account management-policy update --account-name myaccount --resource-group myresourcegroup --add property.listProperty
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could you see the change in response with the new added property?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

could you also add --set examples because customer wants to know how to set the policy as you can see in the issue?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

self.cmd('storage account management-policy update --account-name {sa} -g {rg}'
' --set "policy.rules[0].name=newname"')

@yonzhan yonzhan modified the milestones: S185, S186 Apr 10, 2021
@Juliehzl Juliehzl merged commit bf4df70 into Azure:dev Apr 26, 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.

doc for az storage account management-policy update need example(s)

3 participants