Skip to content

Remove call to metadata API - #47956

Closed
EvertonSA wants to merge 1 commit into
apache:mainfrom
EvertonSA:main
Closed

EvertonSA wants to merge 1 commit into
apache:mainfrom
EvertonSA:main

Conversation

@EvertonSA

@EvertonSA EvertonSA commented Mar 19, 2025 •

Copy link
Copy Markdown
Contributor

closes #47950

Problem: If multiple tasks run in parallel on the same execution instance there is high chance the instance metadata limit is reached as documented here #47950. The call do metadata API was implemented long time ago and today it feels like no other microsoft integration adhered to this strategy (check before call).

This PR removes this behavior and let azure.identity ManagedIdentityCredential class fails naturally.

@EvertonSA
EvertonSA marked this pull request as ready for review March 19, 2025 16:19
@EvertonSA
EvertonSA marked this pull request as draft March 20, 2025 07:04
@EvertonSA

Copy link
Copy Markdown
Contributor Author

converting to draft as tests broke and i also would like a second opinion on this issue

@github-actions

github-actions Bot commented May 5, 2025

Copy link
Copy Markdown
Contributor

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions Bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label May 5, 2025
@github-actions github-actions Bot closed this May 11, 2025
@EvertonSA EvertonSA mentioned this pull request May 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:databricks stale Stale PRs per the .github/workflows/stale.yml policy file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Too many requests

1 participant