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

bump open api image to v7.3.0 #34805

Closed
wants to merge 2 commits into from

Conversation

DrFaust92
Copy link
Contributor

@DrFaust92 DrFaust92 commented Oct 6, 2023

Bump openapi generator image to latest to generate go (and python) clients with newer dependecies (in go case newer go version as today go 1.13 is used and is very old, while new openapi gen uses newer 1.18 image and depencies)


^ 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.

@boring-cyborg boring-cyborg bot added the area:API Airflow's REST/HTTP API label Oct 6, 2023
@DrFaust92 DrFaust92 marked this pull request as ready for review October 6, 2023 14:58
@@ -17,7 +17,7 @@
# specific language governing permissions and limitations
# under the License.

OPENAPI_GENERATOR_CLI_VER=5.4.0
OPENAPI_GENERATOR_CLI_VER=7.0.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will cause a lot of breaking changes to the RESTAPI. We should reserve it for the next major

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am afraid @ephraimbuddy is right.

On the other hand this starts to be a little blocking. For instance for the go client, and other client issues where we need to manually fix things. (Missing supports for certain keywords in the spec etc...).

We didn't discuss this but maybe we could do a major release for clients, independantly of airflow major release, just an idea, to not have clients stuck for a long long time. Like update the client versionning policy to allow that. I don't think major version needs to match between client and airflow core. (It will be a little harder to handle but worth I believe).

If this sounds like an idea worth discussing, I can bring that to the dev list.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pierrejeambrun i think mailing list discussion is required here

Copy link
Member

@potiuk potiuk Oct 22, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah. I think there are also another reason we should discuss it - namely security/release process for the clients. The "Python API" client we release might be seen as an "official ASF Source package"

Currently we are voting on Python Client and we do not check the provenence of it - because this is code that got genereated by the release manager. and committed - unreviewable and unverifiable.

So IMHO upgrading the client is also an opportunity to update the build and release process - what should happen, we (PMC members) should be able to verify that the build has been generated by the release manager from the API specification and not tampered with. It might be a "reproducible" build or just "verifiable build" - but it boils down to being able to run the build and compare if what comes out of it (providing sufficient external tools we can rely-on - such as official image released by open-api).

I think we can solve both problems at the same time - by improving automation and adding simple-to-add verification step whether the package we are voting on is "really" generated from the specification without being tampered with.

This is part of the security improvement of our release process.

@potiuk potiuk mentioned this pull request Oct 29, 2023
1 task
Copy link

github-actions bot commented Dec 7, 2023

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 7, 2023
@github-actions github-actions bot closed this Dec 12, 2023
@ephraimbuddy
Copy link
Contributor

@DrFaust92 , can you rebase

@DrFaust92 DrFaust92 reopened this Dec 19, 2023
@github-actions github-actions bot removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Dec 20, 2023
@DrFaust92 DrFaust92 changed the title bump open api image to v7.0.1 bump open api image to v7.1.0 Dec 23, 2023
Copy link

github-actions bot commented Feb 7, 2024

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Feb 7, 2024
@DrFaust92 DrFaust92 changed the title bump open api image to v7.1.0 bump open api image to v7.3.0 Feb 10, 2024
@github-actions github-actions bot removed the stale Stale PRs per the .github/workflows/stale.yml policy file label Feb 11, 2024
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Mar 30, 2024
@github-actions github-actions bot closed this Apr 4, 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 stale Stale PRs per the .github/workflows/stale.yml policy file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants