Skip to content

Fix deprecation warning on provider lookup in Amazon AWS hooks - #74480

Merged
vincbeck merged 1 commit into
apache:mainfrom
westonkl:fix-amazon-provider-manager-warning
Oct 14, 2026
Merged

vincbeck merged 1 commit into
apache:mainfrom
westonkl:fix-amazon-provider-manager-warning

Conversation

@westonkl

@westonkl westonkl commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Fix deprecated provider lookup in Amazon AWS hooks

AWS hooks emit a DeprecatedImportWarning when adding the provider version to the boto3 user agent because they access ProvidersManager.hooks.

Use ProvidersManagerTaskRuntime when available, retaining the core manager as a fallback for older supported Airflow versions. Add a regression test that promotes the deprecated-hooks warning to an error and verifies the provider version is still returned.


Was generative AI tooling used to co-author this PR?
  • Yes - Codex

@westonkl
westonkl requested a review from o-nikolas as a code owner October 8, 2026 22:25
@boring-cyborg boring-cyborg Bot added area:providers provider:amazon AWS/Amazon - related issues labels Oct 8, 2026
@westonkl westonkl changed the title Fix deprecated provider lookup in Amazon AWS hooks Fix deprecation warning on provider lookup in Amazon AWS hooks Oct 9, 2026
@eladkal
eladkal requested a review from vincbeck October 9, 2026 06:52
@vincbeck

vincbeck commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

@ramitkataria @seanghaeli

@seanghaeli seanghaeli left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, no risk of breaking anything

@seanghaeli

Copy link
Copy Markdown
Contributor

@ferruzzi could you merge this when you get a chance?

@ferruzzi ferruzzi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@vincbeck
vincbeck merged commit 945b794 into apache:main Oct 14, 2026
84 checks passed
@boring-cyborg

boring-cyborg Bot commented Oct 14, 2026

Copy link
Copy Markdown

Awesome work, congrats on your first merged pull request! You are invited to check our Issue Tracker for additional contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants