Skip to content

Remove various upper version bounds in the FAB provider#63868

Merged
potiuk merged 1 commit intoapache:mainfrom
Dev-iL:2603/unlimit_flask
Mar 19, 2026
Merged

Remove various upper version bounds in the FAB provider#63868
potiuk merged 1 commit intoapache:mainfrom
Dev-iL:2603/unlimit_flask

Conversation

@Dev-iL
Copy link
Collaborator

@Dev-iL Dev-iL commented Mar 18, 2026

With uv dependency resolution, upstream limits are automatically enforced. I'm attempting to remove our local limits in case they are outdated (e.g. pre-FAB5) and unnecessary.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {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.

@Dev-iL Dev-iL requested a review from vincbeck as a code owner March 18, 2026 08:32
@Dev-iL Dev-iL mentioned this pull request Mar 18, 2026
6 tasks
@Dev-iL Dev-iL added the full tests needed We need to run full set of tests for this PR to merge label Mar 18, 2026
Copy link
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

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

I do not see any concerns, I'd like to have @potiuk opinion on this as well though

@Dev-iL
Copy link
Collaborator Author

Dev-iL commented Mar 18, 2026

I do not see any concerns, I'd like to have @potiuk opinion on this as well though

My concern is these changes have no effect whatsoever....

@potiuk
Copy link
Member

potiuk commented Mar 19, 2026

My concern is these changes have no effect whatsoever....

Let's get it in then. Once we merge'm we can take a look at our --explain-why reports we have in finalize steps and we will know what else is blocking us.

That's the usual game - remove what's blocking us and find the next obstactle.

Can you please rebase it / fix conflicts?

@Dev-iL
Copy link
Collaborator Author

Dev-iL commented Mar 19, 2026

Sure, just a minute

With uv dependency resolution, upstream limits are automatically enforced.
@Dev-iL Dev-iL force-pushed the 2603/unlimit_flask branch from cc6c6fd to 7ac577e Compare March 19, 2026 11:26
@potiuk potiuk merged commit 91eeb9d into apache:main Mar 19, 2026
129 of 130 checks passed
@Dev-iL Dev-iL deleted the 2603/unlimit_flask branch March 19, 2026 13:11
fat-catTW pushed a commit to fat-catTW/airflow that referenced this pull request Mar 22, 2026
With uv dependency resolution, upstream limits are automatically enforced.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers full tests needed We need to run full set of tests for this PR to merge kind:documentation provider:fab

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants