ML: cli.azure.cli.core.azclierror: Met error <class 'TypeError'>:'SASCredentialDto' object is not subscriptable

[KMoraz]

This is autogenerated. Please review and update as needed.

Describe the bug

Command Name
az ml model download Extension Name: ml. Version: 2.16.0.

Errors:

cli.azure.cli.core.azclierror: Met error <class 'TypeError'>:'SASCredentialDto' object is not subscriptable
az_command_data_logger: Met error <class 'TypeError'>:'SASCredentialDto' object is not subscriptable

To Reproduce:

Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

• az ml model download --name {} --version {} --registry-name {} -w {} --debug

Expected Behavior

Environment Summary

Linux-5.15.90.1-microsoft-standard-WSL2-x86_64-with-glibc2.35, CBL-Mariner/Linux
Python 3.9.14
Installer: RPM

azure-cli 2.48.1

Extensions:
azure-devops 0.26.0
ml 2.16.0

Dependencies:
msal 1.20.0
azure-mgmt-resource 22.0.0

Additional Context

[yonzhan]

Thank you for opening this issue, we will look into it.

[navba-MSFT]

@KMoraz Apologies for the late reply. Thanks for reaching out to us and reporting this issue. Could you please share the--debugoutput while running the above command ?

[KMoraz]

@navba-MSFT sure: ml.out.txt

[navba-MSFT]

@KMoraz Thanks for the debug output. We are looking into this internally.

[eddmik]

I managed to fix this by downgrading the ml extension version by
az extension add --name ml --version 2.15.1

[KMoraz]

@eddmik yes, we also did the same

[uberj]

Also hitting this issue... cannot download models.

[uberj]

Looks like an API was updated to use attribute references rather than dict lookups. Here is a patch to fix the issue for "ml": "2.20.0"

--- /home/uberj/anaconda3/envs/<project>/lib/python3.8/site-packages/azure/ai/ml/_utils/_registry_utils.py     2023-09-22 12:42:24.691203253 -0700
+++ /home/uberj/_registery_utils.py     2023-09-22 12:43:05.731422505 -0700
@@ -173 +173 @@
-    if sas_uri.blob_reference_for_consumption.credential["credentialType"] == "NoCredentials":
+    if sas_uri.blob_reference_for_consumption.credential.credential_type == "NoCredentials":
@@ -176 +176 @@
-    return sas_uri.blob_reference_for_consumption.credential["sasUri"], "SAS"
+    return sas_uri.blob_reference_for_consumption.credential.sas_uri, "SAS"

[uberj]

I just reviewed Azure/azure-sdk-for-python#30395 and it has the same patch. Can it be merged so this can be resolved?

[navba-MSFT]

Service team is involved to look into this.

[monuminu]

Is this issue fixed ?