Skip to content

[28.x backport] api: swagger: Tweak type of GwPriority to integer #50350

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

Merged
merged 2 commits into from
Jul 8, 2025

Conversation

vvoland
Copy link
Contributor

@vvoland vvoland commented Jul 8, 2025

Related to: fussybeaver/bollard#537

- What I did

Changed the GwPriority API property type to "integer".

- How I did it

Modified all API documents in ./docs/api that contain a GwPriority to be an integer. Added a separate commit to modify ./api/swagger.yaml with the same change.

Verified that the types do not change when running ./hack/generate-swagger-api.sh

- How to verify it

Generated the types based on the new API and verified that GwPriority is now an integer.

- Human readable description for the release notes

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Niel Drummond <niel@drummond.lu>
(cherry picked from commit bfc0c7c)
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
Signed-off-by: Niel Drummond <niel@drummond.lu>
(cherry picked from commit 51d6687)
Signed-off-by: Paweł Gronowski <pawel.gronowski@docker.com>
@vvoland vvoland added this to the 28.3.2 milestone Jul 8, 2025
@vvoland vvoland self-assigned this Jul 8, 2025
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@vvoland vvoland merged commit da65c86 into moby:28.x Jul 8, 2025
323 of 329 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants