Skip to content
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

Test fixing bad PROD image preparation in CI #39011

Closed
wants to merge 105 commits into from

Conversation

potiuk
Copy link
Member

@potiuk potiuk commented Apr 14, 2024


^ 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.rst or {issue_number}.significant.rst, in newsfragments.

Ulada Zakharava and others added 30 commits April 14, 2024 15:12
…on dependencies in pyproject.toml.

Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
potiuk and others added 12 commits April 14, 2024 15:12
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Signed-off-by: sudipto baral <sudiptobaral.me@gmail.com>
Switching to flask client rather than starlette, helped to fix the
issue.
The fix checks for the 429 HTTP exception that should be returned
in this case.

This also reverts commit a9aa27d.
Fixed by switching to use flask client for testing rather than
starlette. Starlette client in this case has some side effects that are
also impacting Sqlite's session being created in a different
thread and deleted with close_all_sessions fixture.
Fixed by switching to use flask client for testing rather than
starlette. Starlette client in this case has some side effects that are
also impacting Sqlite's session being created in a different
thread and deleted with close_all_sessions fixture.
Fixed by switching the test to flask_admin_client. Removes
sqlalchemy session creted from a different thread.
@potiuk potiuk added the full tests needed We need to run full set of tests for this PR to merge label Apr 14, 2024
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:CLI area:db-migrations PRs with DB migration area:dev-tools area:providers area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues kind:documentation provider:fab labels Apr 14, 2024
@potiuk potiuk force-pushed the test-swagger-ui-removeavl-connexion3 branch from 5e13957 to 73d7502 Compare April 14, 2024 22:02
@potiuk potiuk force-pushed the test-swagger-ui-removeavl-connexion3 branch from 73d7502 to ae320ca Compare April 14, 2024 22:07
@potiuk potiuk changed the title Test swagger UI removeavl connexion3 Test fixing bad PROD image preparation in CI Apr 15, 2024
@potiuk potiuk added the default versions only When assigned to PR - only default python version is used for CI tests label Apr 15, 2024
When PROD image packages are installed in in CI, the local sources
should not be present in the image, also constraints from sources
shoudl replace the one downloaded from main.
@potiuk potiuk force-pushed the test-swagger-ui-removeavl-connexion3 branch from 5acb266 to 3dbbe4a Compare April 15, 2024 15:03
@potiuk potiuk closed this Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:API Airflow's REST/HTTP API area:CLI area:db-migrations PRs with DB migration area:dev-tools area:providers area:UI Related to UI/UX. For Frontend Developers. area:webserver Webserver related Issues default versions only When assigned to PR - only default python version is used for CI tests 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.

None yet

4 participants