Skip to content

Fix managed-identity notebook build failure from list_keys API shape change - #3948

Merged
Chakradhar886 merged 4 commits into
mainfrom
copilot/fix-github-actions-build-job-again
May 27, 2026
Merged

Fix managed-identity notebook build failure from list_keys API shape change#3948
Chakradhar886 merged 4 commits into
mainfrom
copilot/fix-github-actions-build-job-again

Conversation

Copilot AI commented May 22, 2026

Copy link
Copy Markdown
Contributor

Error:
The notebook failed while reading the storage account key after list_keys, because the code assumed one response shape and tried attribute-style access only.

Why it was failing:

1.Runtime dependency resolution changed over time (these notebooks install prerelease SDKs).
2.The same list_keys call can come back in different valid shapes depending on SDK/runtime combination.
3.Old logic handled only one shape, so it failed when the alternate shape appeared.

What we fixed:
1.Added compatibility handling for both response shapes (model-style and mapping-style).
2Normalized key extraction so value is read correctly in either case.
3.Kept behavior unchanged: still uses the first returned storage key.

Net result:
No functional workflow change, only robustness fix to prevent environment/version-related breakage.

Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'build' Fix managed-identity notebook build failure from list_keys API shape change May 22, 2026
Copilot AI requested a review from Chakradhar886 May 22, 2026 05:16
@Chakradhar886
Chakradhar886 marked this pull request as ready for review May 22, 2026 06:34
@Chakradhar886
Chakradhar886 requested a review from a team as a code owner May 22, 2026 06:34
@Chakradhar886

Chakradhar886 commented May 26, 2026

Copy link
Copy Markdown
Member

Description

Error:
The notebook failed while reading the storage account key after list_keys, because the code assumed one response shape and tried attribute-style access only.

Why it was failing:

1.Runtime dependency resolution changed over time (these notebooks install prerelease SDKs).
2.The same list_keys call can come back in different valid shapes depending on SDK/runtime combination.
3.Old logic handled only one shape, so it failed when the alternate shape appeared.

What we fixed:
1.Added compatibility handling for both response shapes (model-style and mapping-style).
2Normalized key extraction so value is read correctly in either case.
3.Kept behavior unchanged: still uses the first returned storage key.

Net result:
No functional workflow change, only robustness fix to prevent environment/version-related breakage.

@lgayhardt lgayhardt left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues for the docs =)

@Chakradhar886
Chakradhar886 merged commit 0c08e2f into main May 27, 2026
15 checks passed
@Chakradhar886
Chakradhar886 deleted the copilot/fix-github-actions-build-job-again branch May 27, 2026 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants