Eager upgrade works also in editable mode. #13589
Conversation
|
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 master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
github-actions
bot
added
the
full tests needed
potiuk
force-pushed
the
fix-failing-master-on-prod-image-build
branch
from
0d799d5 to
822bc4a
Compare
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
potiuk
force-pushed
the
fix-failing-master-on-prod-image-build
branch
2 times, most recently
from
9ca9871 to
c7d014a
Compare
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
potiuk
force-pushed
the
fix-failing-master-on-prod-image-build
branch
from
c7d014a to
d09caf7
Compare
|
The Workflow run is cancelling this PR. Building images for the PR has failed. Follow the the workflow link to check the reason. |
The recent change apache#13409 introduced common installation script but one case turned out to be not working. Installing Airflow in editable mode with eager upgrade fails on building production image (the CI image effectively did not have airflow installed in editable mode, which it should). This PR fixes that.
potiuk
force-pushed
the
fix-failing-master-on-prod-image-build
branch
from
d09caf7 to
1f1a56e
Compare
|
Looks like the problem is fixed. Merging :) |
The recent change #13409 introduced common installation script but one case turned out to be not working. Installing Airflow in editable mode with eager upgrade fails on building production image (the CI image effectively did not have airflow installed in editable mode, which it should). This PR fixes that. (cherry picked from commit 543194d)
2 participants
The recent change #13409 introduced common installation script
but one case turned out to be not working. Installing Airflow
in editable mode with eager upgrade fails on building production
image (the CI image effectively did not have airflow installed
in editable mode, which it should).
This PR fixes that.
^ 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.