Skip to content

[PULP-820][3.32] Downgrade openapi-spec version to 3.0.1#4134

Merged
pedro-psb merged 1 commit into
pulp:3.32from
pedro-psb:3.32/downgrade-openapi-version
Sep 18, 2025
Merged

[PULP-820][3.32] Downgrade openapi-spec version to 3.0.1#4134
pedro-psb merged 1 commit into
pulp:3.32from
pedro-psb:3.32/downgrade-openapi-version

Conversation

@pedro-psb

@pedro-psb pedro-psb commented Sep 17, 2025

Copy link
Copy Markdown
Member

Fixes generation of ruby binding reported in #4125

@pedro-psb

Copy link
Copy Markdown
Member Author

Tested the generated client from the current PR against the test proposed here (see the last commit for the test). The conclusion is that it indeed fixes the issue.

You can try it yourself syncing from a modular pulp fixtures repository and trying to list it using (a) published ruby bindings (should fail) (b) ruby bindings from current PR (should succeed). Refer to the test mentioned above for details of this reproducer.

@pedro-psb pedro-psb marked this pull request as ready for review September 17, 2025 22:11
@pedro-psb pedro-psb force-pushed the 3.32/downgrade-openapi-version branch from 51839fd to a965e64 Compare September 17, 2025 22:13
@pedro-psb pedro-psb changed the title Downgrade openapi-spec version to 3.0.1 [PULP-820] Downgrade openapi-spec version to 3.0.1 Sep 17, 2025
@dralley

dralley commented Sep 17, 2025

Copy link
Copy Markdown
Contributor

Is this a test, test, or is it mergable?

@pedro-psb

Copy link
Copy Markdown
Member Author

It's mergeable. The manual tests mentioned reveals it works.

@pedro-psb pedro-psb changed the title [PULP-820] Downgrade openapi-spec version to 3.0.1 [PULP-820][3.32] Downgrade openapi-spec version to 3.0.1 Sep 18, 2025
@pedro-psb pedro-psb enabled auto-merge (rebase) September 18, 2025 12:32
@pedro-psb pedro-psb linked an issue Sep 18, 2025 that may be closed by this pull request
@pedro-psb pedro-psb merged commit f197166 into pulp:3.32 Sep 18, 2025
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

uninitialized constant PulpRpmClient::OneOf when listing ModuleMds

2 participants