Uh oh!
There was an error while loading. Please reload this page.
Resolve nested provider paths for conn-fields check from the real directory tree - #70261
Conversation
fdde0d6 to
8e476faCompare8e476fa to
54d0e6dCompareUh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
54d0e6d to
6066ad7Compare…ectory tree The conn-fields prek hook guessed how many path segments made up a provider package name from a hardcoded list of known namespace prefixes, so it kept crashing on newly nested providers (e.g. IBM MQ) until someone manually patched the list. Walking the actual directory tree to find each provider's provider.yaml makes it work for any current or future nested provider automatically.
6066ad7 to
cee89ccCompareUh oh!
There was an error while loading. Please reload this page.
Backport failed to create: v3-3-test. View the failure log Run detailsNote: As of Merging PRs targeted for Airflow 3.X In matter of doubt please ask in #release-management Slack channel.
You can attempt to backport this manually by running: cherry_picker 030efa6 v3-3-testThis should apply the commit to the v3-3-test branch and leave the commit in conflict state marking After you have resolved the conflicts, you can continue the backport process by running: cherry_picker --continueIf you don't have cherry-picker installed, see the installation guide. |
Why
#70224 (comment)
The conn-fields prek hook guessed how many path segments made up a provider package name from a hardcoded list of known namespace prefixes, so it kept crashing on newly nested providers (e.g. IBM MQ) until someone manually patched the list.
What
Walking the actual directory tree to find each provider's provider.yaml makes it work for any current or future nested provider automatically.
Was generative AI tooling used to co-author this PR?
Generated-by: [Claude] following the guidelines
{pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.