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

Update VM api version and add "placement" into diffDiskSettings #1778

Merged
merged 1 commit into from
Mar 4, 2022

Conversation

lubaihua33
Copy link
Collaborator

@lubaihua33 lubaihua33 commented Mar 4, 2022

For ephemeral disk, if you want to opt for OS cache placement, the image OS disk's size should be less than or equal to the cache size of the VM size chosen. See doc.

When the image is Ubuntu20.04 gen2, and the VM size is D96ds_v5, from the Azure portal, it shows "The selected image is too large for the OS cache of the selected instance." The OS cache placement option is not available.

image

In lisav2, there is no "placement" specified in "diffDiskSettings". The VM is deployed as Temp disk placement.

But for SIG, it has no information on the image size. The error "An error occurred while loading image/size information. Ephemeral OS disk options are available, but they may fail to deploy" shows in the Azure portal. The OS cache placement is available. When using this template, the VM is deployed as cache disk type and the provision is failed with the errors "OS disk of Ephemeral VM with size greater than 3 GB is not allowed for VM size when the DiffDiskPlacement is CacheDisk".

image

When we add ""placement": "ResourceDisk" " in "diffDiskSettings" object, it has error "Could not find member 'placement' on object of type 'DiffDiskSettings'". After updating the API version to "2020-12-01". The SIG image can be deployed with the ephemeral disk.

@squirrelsc squirrelsc merged commit fe010a3 into main Mar 4, 2022
@squirrelsc squirrelsc deleted the baihua/lisav3_disk branch March 4, 2022 16:35
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.

2 participants