Skip to content

Commit 3ae6e8b

Browse files
authored
fixing data store connection issue (#38635)
* fixing data store connection issue * updating version file
1 parent 3e52c6c commit 3ae6e8b

3 files changed

Lines changed: 9 additions & 9 deletions

File tree

sdk/ml/azure-ai-ml/CHANGELOG.md

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,9 @@
11
# Release History
22

3-
## 1.22.2 (2024-11-18)
3+
## 1.22.4 (2024-11-21)
44

55
### Bugs Fixed
6-
- **#3620407** - Fix Datastore credentials show up as NoneCredentials
7-
8-
## 1.22.1 (2024-11-13)
9-
10-
### Bugs Fixed
11-
- **#38493** - Fix error NoneType object is not subscriptable
6+
- **#38609** - Fix Datastore credentials for Account key base
127

138
## 1.22.3 (2024-11-20)
149

sdk/ml/azure-ai-ml/azure/ai/ml/_version.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,4 @@
22
# Copyright (c) Microsoft Corporation. All rights reserved.
33
# ---------------------------------------------------------
44

5-
VERSION = "1.22.3"
5+
VERSION = "1.22.4"

sdk/ml/azure-ai-ml/azure/ai/ml/entities/_datastore/utils.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,12 @@ def from_rest_datastore_credentials(
3030
config_class: Any = NoneCredentialConfiguration
3131

3232
if isinstance(rest_credentials, (models.AccountKeyDatastoreCredentials, models2024.AccountKeyDatastoreCredentials)):
33-
config_class = AccountKeyConfiguration
33+
# we are no more using key for key base account.
34+
# https://github.com/Azure/azure-sdk-for-python/pull/35716
35+
if isinstance(rest_credentials.secrets, models2024.SasDatastoreSecrets):
36+
config_class = SasTokenConfiguration
37+
else:
38+
config_class = AccountKeyConfiguration
3439
elif isinstance(rest_credentials, (models.SasDatastoreCredentials, models2024.SasDatastoreCredentials)):
3540
config_class = SasTokenConfiguration
3641
elif isinstance(

0 commit comments

Comments
 (0)