Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Apr 15, 2025

When redoc includes, rather than embeds the openapi schemes, it loads them from the same folder. If the files have the same names they will be overwritten.

This PR changes the openapi schemes to all have distinct, unique names.


^ 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 airflow-core/newsfragments.

When redoc includes, rather than embeds the openapi schemes, it
loads them from the same folder. If the files have the same names
they will be overwritten.

This PR changes the openapi schemes to all have distinct, unique
names.
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:dev-tools area:providers area:UI Related to UI/UX. For Frontend Developers. kind:documentation provider:fab labels Apr 15, 2025
@potiuk potiuk requested review from amoghrajesh and kaxil April 15, 2025 12:42
@potiuk potiuk changed the title Make sure all openapi schemes have distinct names Make sure all openapi schemas have distinct names Apr 15, 2025
@potiuk
Copy link
Member Author

potiuk commented Apr 15, 2025

Tests failing with main broken in #49239

@potiuk potiuk merged commit 90dbd24 into apache:main Apr 15, 2025
92 of 99 checks passed
@potiuk potiuk deleted the make-sure-openapi-generated-specs-have-different-names branch April 15, 2025 13:30
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:dev-tools area:providers area:UI Related to UI/UX. For Frontend Developers. kind:documentation provider:fab

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants