MicrosoftDocs / azure-docs

Open source documentation of Microsoft Azure
https://docs.microsoft.com/azure
Creative Commons Attribution 4.0 International
10.12k stars 21.18k forks source link

Rethink case for '--extension-type Microsoft.AzureML.Kubernetes' #110642

Open anwojcie opened 1 year ago

anwojcie commented 1 year ago

Hi,

all the example az cli commands use '--extension-type Microsoft.AzureML.Kubernetes' When trying to add a Kubernetes cluster in the AML Webinterface, the available cluster for the drop down are looked up by running a resource graph query:

kubernetesconfigurationresources
      | where type == "microsoft.kubernetesconfiguration/extensions"
      | where properties.ExtensionType == "microsoft.azureml.kubernetes"
      | extend cluster_id = substring(id, 0, indexof(id, "/providers/Microsoft.KubernetesConfiguration/Extensions"))
      | extend cluster_name = substring(cluster_id, strlen(cluster_id) - indexof(reverse(cluster_id), "/"))
      | extend version = properties.Version
      | project cluster_id, resourceGroup, version, cluster_name
      | sort by (tolower(cluster_name)) asc

note: where type == "microsoft.kubernetesconfiguration/extensions" is case sensitive and won't find and offer the cluster with an extension created by the commands in this page.

Either we need to change the resource graph query in AML

kubernetesconfigurationresources
-      | where type == "microsoft.kubernetesconfiguration/extensions"
+      | where type =~ "microsoft.kubernetesconfiguration/extensions"
      | where properties.ExtensionType == "microsoft.azureml.kubernetes"
      | extend cluster_id = substring(id, 0, indexof(id, "/providers/Microsoft.KubernetesConfiguration/Extensions"))
      | extend cluster_name = substring(cluster_id, strlen(cluster_id) - indexof(reverse(cluster_id), "/"))
      | extend version = properties.Version
      | project cluster_id, resourceGroup, version, cluster_name
      | sort by (tolower(cluster_name)) asc

Or every page in our docs which uses Microsoft.AzureML.Kubernetes

Cheers, Andre


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

RamanathanChinnappan-MSFT commented 1 year ago

@anwojcie

Thanks for your feedback! We will investigate and update as appropriate.

RamanathanChinnappan-MSFT commented 1 year ago

@anwojcie

I've delegated this to @Bozhong68, a content author, to review and share their valuable insights.