Skip to content

Commit 0890873

Browse files
committed
fix: tidy up command description
1 parent 59dec35 commit 0890873

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

src/aks-preview/azext_aks_preview/_params.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3215,6 +3215,9 @@ def load_arguments(self, _):
32153215
action="store_true",
32163216
help="Show only VM SKUs that support availability zones.",
32173217
)
3218+
# TODO: Eventually deprecate the -all param.
3219+
# The List VM SKUs API already performs regional filtering so once AZ filtering is also implemented
3220+
# within the API, this param will no longer be required.
32183221
c.argument(
32193222
"show_all",
32203223
options_list=["--all"],

src/aks-preview/azext_aks_preview/custom.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3436,10 +3436,9 @@ def aks_get_versions(cmd, client, location): # pylint: disable=unused-argumen
34363436

34373437

34383438
def aks_list_vm_skus(cmd, client, location, size=None, zone=None, show_all=None): # pylint: disable=unused-argument
3439-
"""List VM SKUs available for AKS node pools in a given location.
3439+
"""Lists the VM SKUs accepted by AKS when creating node pools in a specified location.
34403440
3441-
Mirrors the shape and filtering behavior of 'az vm list-skus', but queries the
3442-
AKS-specific endpoint so only SKUs accepted by AKS are returned.
3441+
AKS will perform a best effort approach to provision the requested VM SKUs, but availability is not guaranteed.
34433442
34443443
:param location: Azure region to query.
34453444
:param size: Optional partial VM size name filter (case-insensitive).

0 commit comments

Comments
 (0)