Documentation and tests

Author: atombrella

.web-docs/components/builder/arm/README.md

@@ -290,7 +290,7 @@ Providing `temp_resource_group_name` or `location` in combination with
   learn more about managed images.
 
 - `managed_image_storage_account_type` (string) - Specify the storage account
-  type for a managed image. Valid values are Standard_LRS and Premium_LRS.
+  type for a managed image. Valid values are Standard_LRS, Premium_LRS, StandardSSD_LRS, UltraSSD_LRS, Premium_ZRS,StandardSSD_ZRS and PremiumV2_LRS.
   The default is Standard_LRS.
 
 - `managed_image_os_disk_snapshot_name` (string) - If

builder/azure/arm/config.go

@@ -119,7 +119,8 @@ type SharedImageGalleryDestination struct {
 	// See the `shared_image_gallery_destination` block description for an example of using this field
 	SigDestinationTargetRegions []TargetRegion `mapstructure:"target_region"`
 	// Specify a storage account type for the Shared Image Gallery Image Version.
-	// Defaults to `Standard_LRS`. Accepted values are `Standard_LRS`, `Standard_ZRS` and `Premium_LRS`
+	// Defaults to `Standard_LRS`. Accepted values are `Standard_LRS`, `Standard_ZRS` and `Premium_LRS`, `PremiumV2_LRS`,
+	// `StandardSSD_LRS`, `StandardSSD_ZRS`, `UltraSSD_LRS`
 	SigDestinationStorageAccountType string `mapstructure:"storage_account_type"`
 	// Set to true if publishing to a Specialized Gallery, this skips a call to set the build VM's OS state as Generalized
 	SigDestinationSpecialized bool `mapstructure:"specialized"`
@@ -1552,6 +1553,14 @@ func assertRequiredParametersSet(c *Config, errs *packersdk.MultiError) {
 		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardLRS
 	case string(virtualmachines.StorageAccountTypesPremiumLRS):
 		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesPremiumLRS
+	case string(virtualmachines.StorageAccountTypesPremiumVTwoLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesPremiumVTwoLRS
+	case string(virtualmachines.StorageAccountTypesStandardSSDLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardSSDLRS
+	case string(virtualmachines.StorageAccountTypesStandardSSDZRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardSSDZRS
+	case string(virtualmachines.StorageAccountTypesUltraSSDLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesUltraSSDLRS
 	default:
 		errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("The managed_image_storage_account_type %q is invalid", c.ManagedImageStorageAccountType))
 	}

builder/azure/dtl/config.go

@@ -210,8 +210,9 @@ type Config struct {
 	// learn more about managed images.
 	ManagedImageName string `mapstructure:"managed_image_name" required:"true"`
 	// Specify the storage account
-	// type for a managed image. Valid values are Standard_LRS and Premium_LRS.
-	// The default is Standard_LRS.
+	// type for a managed image. Valid values are Standard_LRS, Premium_LRS,
+	// StandardSSD_LRS, StandardSSD_ZRS, PremiumV2_LRS, PremiumV2_ZRS, UltraSSD_LRS
+	// The default is Standard_LRS.'
 	ManagedImageStorageAccountType string `mapstructure:"managed_image_storage_account_type" required:"false"`
 	managedImageStorageAccountType virtualmachines.StorageAccountTypes
 
@@ -747,6 +748,14 @@ func assertRequiredParametersSet(c *Config, errs *packersdk.MultiError) {
 		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardLRS
 	case string(images.StorageAccountTypesPremiumLRS):
 		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesPremiumLRS
+	case string(images.StorageAccountTypesPremiumVTwoLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesPremiumVTwoLRS
+	case string(images.StorageAccountTypesStandardSSDLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardSSDLRS
+	case string(images.StorageAccountTypesStandardSSDZRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesStandardSSDZRS
+	case string(images.StorageAccountTypesUltraSSDLRS):
+		c.managedImageStorageAccountType = virtualmachines.StorageAccountTypesUltraSSDLRS
 	default:
 		errs = packersdk.MultiErrorAppend(errs, fmt.Errorf("The managed_image_storage_account_type %q is invalid", c.ManagedImageStorageAccountType))
 	}

builder/azure/dtl/config_test.go

@@ -409,7 +409,15 @@ func TestConfigShouldAcceptManagedImageStorageAccountTypes(t *testing.T) {
 		"os_type": constants.Target_Linux,
 	}
 
-	storage_account_types := []string{"Premium_LRS", "Standard_LRS"}
+	storage_account_types := []string{
+		"Premium_LRS",
+		"PremiumV2_LRS",
+		"Premium_ZRS",
+		"Standard_LRS",
+		"StandardSSD_LRS",
+		"StandardSSD_ZRS",
+		"UltraSSD_LRS",
+	}
 
 	for _, x := range storage_account_types {
 		config_map["managed_image_storage_account_type"] = x