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
Fix session usage in /rendered-k8s view #21006
Conversation
We can't commit the session too early because later functions need that session to fetch related objects.
Hmm, not sure what’s wrong with the doc build job. |
I tihnk it was an intermittent problem with trying to reach the intersphinx inventory. |
Yep. All good now. |
The PR is likely OK to be merged with just subset of tests for default Python and Database versions without running the full matrix of tests, because it does not modify the core of Airflow. If the committers decide that the full tests matrix is needed, they will add the label 'full tests needed'. Then you should rebase to the latest main or amend the last commit of the PR, and push it with --force-with-lease. |
We can't commit the session too early because later functions need that
session to fetch related objects.
Fix #20534.