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

[Compute] Upgrade disk API version to 2020-06-30 and some new features #15031

Merged
merged 11 commits into from
Sep 10, 2020

Conversation

qwordy
Copy link
Member

@qwordy qwordy commented Sep 3, 2020

Description

Resolve #14899

[Compute] Upgrade disk API version to 2020-06-30

[Compute] disk create: add --logical-sector-size, --tier
Logical sector size in bytes for Ultra disks. Supported values are 512 ad 4096. 4096 is the default.

[Compute] disk update: Support --disk-iops-read-only, --disk-mbps-read-only, --max-shares

[Compute] new command disk-encryption-set list-associated-resources

Testing Guide

History Notes

[Compute] Upgrade disk API version to 2020-06-30
[Compute] disk create: add --logical-sector-size, --tier
[Compute] disk update: Support --disk-iops-read-only, --disk-mbps-read-only, --max-shares
[Compute] new command disk-encryption-set list-associated-resources


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

@yonzhan
Copy link
Collaborator

yonzhan commented Sep 4, 2020

Compute

@qwordy qwordy requested a review from v-Ajnava as a code owner September 8, 2020 02:52
@qwordy qwordy changed the title [Compute] Upgrade disk API version to 2020-06-30 [Compute] Upgrade disk API version to 2020-06-30 and some new features Sep 8, 2020
@qwordy
Copy link
Member Author

qwordy commented Sep 9, 2020

Create an issue to track failed tests.
#15097

@@ -134,8 +134,8 @@ def default_api_version(self):
ResourceType.MGMT_NETWORK: '2020-05-01',
ResourceType.MGMT_COMPUTE: SDKProfile('2020-06-01', {
'resource_skus': '2019-04-01',
'disks': '2020-05-01',
Copy link
Contributor

Choose a reason for hiding this comment

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

is 2020-06-1 api-version available on all regions including sovereign cloud, e.g. Azure China?
@VybavaRamadoss

Copy link
Contributor

Choose a reason for hiding this comment

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

/hold
hold on a moment for confirm, thanks.

Copy link
Member

@jiasli jiasli left a comment

Choose a reason for hiding this comment

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

Core LGTM

Copy link
Contributor

@haroldrandom haroldrandom left a comment

Choose a reason for hiding this comment

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

Network is ok from my side

@qwordy
Copy link
Member Author

qwordy commented Sep 9, 2020

@v-Ajnava Can you review this PR? You are code owner.

@yungezz yungezz added the Compute az vm/vmss/image/disk/snapshot label Sep 9, 2020
Copy link
Member

@v-Ajnava v-Ajnava left a comment

Choose a reason for hiding this comment

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

LGTM

@qwordy qwordy merged commit 31edf3b into Azure:dev Sep 10, 2020
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.

DiskRP 2020-06-30 CLI changes
10 participants