-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
api/v1: update swagger to v0.20.1 #9444
Conversation
test-me-please |
15df65b
to
844541b
Compare
Signed-off-by: André Martins <andre@cilium.io>
844541b
to
f82573e
Compare
test-me-please |
@aanm are there any hand-written changes in this PR, or is it fully generated? |
@@ -16,7 +16,7 @@ BPF_FILES_EVAL := $(shell git ls-files $(ROOT_DIR)/bpf/ | grep -v .gitignore | t | |||
BPF_FILES ?= $(BPF_FILES_EVAL) | |||
BPF_SRCFILES := $(subst ../,,$(BPF_FILES)) | |||
|
|||
SWAGGER_VERSION := v0.19.0 | |||
SWAGGER_VERSION := v0.20.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Only this one @nebril , the rest was all auto generated. I mean there were some necessary refactoring in our code because swagger changed the name of IPAM
to Ipam
.
api.IPAMPostIPAMHandler = NewPostIPAMHandler(d) | ||
api.IPAMPostIPAMIPHandler = NewPostIPAMIPHandler(d) | ||
api.IPAMDeleteIPAMIPHandler = NewDeleteIPAMIPHandler(d) | ||
api.IpamPostIpamHandler = NewPostIPAMHandler(d) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fact that a code generator which deals with APIs thinks that changing case on APIs is OK is absurd to me... sigh.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Please release note this casing change for any users of the Cilium API as if they update to a new version of Cilium in say a go.mod
and try to access our IPAM types, the casing will be different.
Signed-off-by: André Martins andre@cilium.io
This change is