Improve CI test that prepares and test python client #44007
Conversation
potiuk
requested review from
dstandish,
eladkal,
kaxil,
bbovenzi,
gopidesupavan,
ephraimbuddy,
pierrejeambrun,
romsharon98 and
jedcunningham
potiuk
force-pushed
the
improve-openapi-client-tests
branch
2 times, most recently
from
9fc3659
to
fe7f0e0
Compare
The test for python client require locally installed airflow with some providers prepared from sources. We have been preparing the selected packages and installing them, but sometimes this required to add new package there. To make it simpler, we can instead prepare and install all the packages - this will be pretty fast with `uv` and we will avoid to manually modify the CI tests every time we add some package that requires installation.
potiuk
force-pushed
the
improve-openapi-client-tests
branch
from
fe7f0e0
to
9138f5f
Compare
|
Actually - I've hit the same problem that made the breeze ci image necessary - hard to install Airflow with all dependencies in consistent way in CI (uv wants to build kerberos :). I think I will actually change it to run the client inside breeze - but I will defer it after coming back from Ireland (I am going to be far less available next week). |
|
converted to draft for now |
|
I will create an issue for that maybe @gopidesupavan or others would like to take that :) |
|
I added #44020 describing what needs to be done |
Cool , Interesting one to solve :) |
Indeed :). It touches a bit of everything :D |
The test for python client require locally installed airflow with some providers prepared from sources. We have been preparing the selected packages and installing them, but sometimes this required to add new package there.
To make it simpler, we can instead prepare and install all the packages - this will be pretty fast with
uvand we will avoid to manually modify the CI tests every time we add some package that requires installation.^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an 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 a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.