-
Notifications
You must be signed in to change notification settings - Fork 16.3k
Update FAB to latest released from 3.4 line (3.4.5) #22596
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
aaaah |
|
Bug in new breeze 😱 after merge |
458505d to
630b1ac
Compare
|
Rebased after fix in #22597 (bad rebase for Breee2 change) |
We checked that the changes introduced between 3.4.4 and 3.4.5 do not require from us to change the vendored-in security manager.
630b1ac to
ada035b
Compare
|
Just checking - seems the failure was due to bad merge already fixed in #22592 but better to be safe than sorry |
|
The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
We checked that the changes introduced between 3.4.4 and 3.4.5 do not require from us to change the vendored-in security manager. (cherry picked from commit d4846e4)
We checked that the changes introduced between 3.4.4 and 3.4.5 do not require from us to change the vendored-in security manager. (cherry picked from commit d4846e4)
We checked that the changes introduced between 3.4.4 and 3.4.5
do not require from us to change the vendored-in security manager.
^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.