Can't create DataDisk with compute.CachingTypesReadWrite or compute.StorageAccountTypesStandardSSDLRS

[nyetwurk]

Bug Report

github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2018-10-01/compute

github.com/Azure/azure-sdk-for-go v40.6.0+incompatible

go1.14.6

	vm.StorageProfile.DataDisks = &[]compute.DataDisk{{
		Lun:          to.Int32Ptr(0),
		Name:         to.StringPtr(diskName),
		CreateOption: compute.DiskCreateOptionTypesEmpty,
		DiskSizeGB:   to.Int32Ptr(sizeGB),
		Caching:      compute.CachingTypesReadWrite, // doesn't work
		ManagedDisk: &compute.ManagedDiskParameters{ // doesn't work
			StorageAccountType: compute.StorageAccountTypesStandardSSDLRS,
			ID:                 &id,
		},
	}}

I still end up with CachingTypesNone and StandardHDD

[nyetwurk]

I'm guessing those parameters are read only and have to create a storage account first with storageAccountsClient.Create()?

[ghost]

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @Drewm3, @avirishuv.

Issue Details

---

Bug Report

"github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2018-10-01/compute"

github.com/Azure/azure-sdk-for-go v40.6.0+incompatible

go1.14.6

	vm.StorageProfile.DataDisks = &[]compute.DataDisk{{
		Lun:          to.Int32Ptr(0),
		Name:         to.StringPtr(diskName),
		CreateOption: compute.DiskCreateOptionTypesEmpty,
		DiskSizeGB:   to.Int32Ptr(sizeGB),
		Caching:      compute.CachingTypesReadWrite, // doesn't work
		ManagedDisk: &compute.ManagedDiskParameters{ // doesn't work
			StorageAccountType: compute.StorageAccountTypesStandardSSDLRS,
			ID:                 &id,
		},
	}}

I still end up with CachingTypesNone and StandardHDD

Author:	nyetwurk
Assignees:	-
Labels:	question, Compute - VM, Service Attention, Mgmt, customer-reported
Milestone:	-

[jaylabell]

@Drewm3 Do you mind taking a look at this issue please?

[Drewm3]

Hi @nyetwurk, I apologize that this issue was not triaged over to the correct team for over a year.

I am not an expert in the Go SDK, but I see that I am able to create a VM through the Azure Portal and Rest APIs that has a StandardSSD data disk with the caching set to ReadWrite caching. Could you attempt to recreate the issue using the current API version (2021-07-01) to see if the problem still occurs? If the problem does still occur with the current SDK, then I will need to find someone with Go expertise to help debug why it is not working for you or if there is some weird issue that is only showing up in the Go SDK instead of the backend service.

[nyetwurk]

I will test this time permitting. We have mostly abandoned this project partially due to this issue, but will revisit this if the issue is resolved. Is there documentation that confirms this is the correct way to instantiate an SSD drive?

That is to say, storageAccountsClient.Create() is not needed, only that storageAccountType is set in vm.StorageProfile.DataDisks? Because the latter method appears to be read-only.

[Drewm3]

Correct, you only need to set the storage account type in the VM create. No storageAccountClient.Create() is needed.