Skip to content

Commit d8a92f6

Browse files
authored
[AKS] az aks create: Fix the issue where enabling v1 container storage does not fail if the VM SKU field is left empty (#32109)
1 parent fe51337 commit d8a92f6

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/azure-cli/azure/cli/command_modules/acs/azurecontainerstorage/_validators.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -661,7 +661,7 @@ def _validate_nodepools( # pylint: disable=too-many-branches,too-many-locals
661661
'and retry the Azure Container Storage operation.'
662662
)
663663
vm_size = agentpool.get("vm_size")
664-
if vm_size is not None:
664+
if vm_size is not None and vm_size != "":
665665
cpu_value, nvme_enabled = get_vm_sku_details(vm_size.lower())
666666
if cpu_value is None or nvme_enabled is None:
667667
raise UnknownError(

src/azure-cli/azure/cli/command_modules/acs/managed_cluster_decorator.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8825,7 +8825,7 @@ def update_azure_container_storage(self, mc: ManagedCluster) -> ManagedCluster:
88258825
except Exception as ex:
88268826
raise UnknownError(
88278827
f"An error occurred while checking the version of Azure Container Storage"
8828-
f"extension installed on the cluster: {str(ex)}"
8828+
f" extension installed on the cluster: {str(ex)}"
88298829
) from ex
88308830

88318831
disable_azure_container_storage_v1 = disable_azure_container_storage_param is not None and is_container_storage_v1_extension_installed

0 commit comments

Comments
 (0)