-
Notifications
You must be signed in to change notification settings - Fork 39.8k
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
add GRS, RAGRS storage account type support for azure disk #55931
Conversation
/unassign |
@jsafrane: GitHub didn't allow me to assign the following users: brendanburns. Note that only kubernetes members can be assigned. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
@colemickens, can you please add more azure-aware people to OWNERS as reviewers? I've no clue what GRS means. |
(Brendan's username has the cc: @jdumars @seanknox @brendandburns @jackfrancis Not sure who is doing upstream work right now. Can whoever is go ahead and place themselves in the OWNERS file? As FYI: "GRS" is "Geographically redundant storage". Azure Storage has different SKUs with different replication attributes, etc. |
@andyzhangx: GitHub didn't allow me to assign the following users: brendanburns. Note that only kubernetes members can be assigned. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/assign @brendandburns |
Thx, I will put my name in the azure reviewer list. |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: andyzhangx, rootfs Associated issue: 55774 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
Automatic merge from submit-queue (batch tested with PRs 55963, 55790, 55670, 55931). If you want to cherry-pick this change to another branch, please follow the instructions here. |
What this PR does / why we need it:
add Standard_GRS, Standard_RAGRS storage account type support for azure disk, and Standard_ZRS, Premium_GRS are not supported for azure disk now
Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #55774
Special notes for your reviewer:
Release note:
/sig azure
@rootfs @brendanburns