-
Notifications
You must be signed in to change notification settings - Fork 38.7k
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
eliminate dependency on go-openapi/spec #101234
eliminate dependency on go-openapi/spec #101234
Conversation
Hi @gautierdelorme. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
/test pull-kubernetes-integration flake #100556 |
/test pull-kubernetes-e2e-kind flake #101275 |
Signed-off-by: Gautier Delorme <gautier.delorme@gmail.com>
7174339
to
076ed66
Compare
/priority important-soon |
Signed-off-by: Gautier Delorme <gautier.delorme@gmail.com>
076ed66
to
34b0fce
Compare
/test pull-kubernetes-integration flake #100556 |
/test pull-kubernetes-e2e-gce-ubuntu-containerd |
thanks @gautierdelorme assigning to the right folks. looks very mechanical so i am happy to lgtm after approval |
@@ -419,7 +419,6 @@ github.com/go-openapi/jsonreference | |||
# github.com/go-openapi/loads => github.com/go-openapi/loads v0.19.4 | |||
# github.com/go-openapi/runtime => github.com/go-openapi/runtime v0.19.4 | |||
# github.com/go-openapi/spec v0.19.5 => github.com/go-openapi/spec v0.19.5 | |||
## explicit | |||
github.com/go-openapi/spec |
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.
we didn't actually drop anything under vendor/github.com/go-openapi/spec
as part of this, so we actually increased our dependencies... what is requiring github.com/go-openapi/spec
now?
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.
@liggitt this is what I shared in Slack:
github.com/go-openapi/spec
is still being pulled in by kustomize
though (but k/k doesn't depend on it anymore).
$ go mod why github.com/go-openapi/spec
# github.com/go-openapi/spec
k8s.io/kubernetes/pkg/kubectl/cmd/convert
k8s.io/cli-runtime/pkg/resource
sigs.k8s.io/kustomize/api/krusty
sigs.k8s.io/kustomize/kyaml/openapi
github.com/go-openapi/spec
Not sure if that's something we want to change in the future (https://github.com/kubernetes-sigs/kustomize/blob/f93cee94404bf0b60ff82785be9ff2af9797463f/kyaml/openapi/openapi.go#L14).
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.
ah, right... #98946 (comment)
tracked in kubernetes-sigs/kustomize#3705
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: gautierdelorme, liggitt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
generated openapi files still reference this, xref #101375 |
What type of PR is this?
/kind cleanup
What this PR does / why we need it:
We don't need to rely on
github.com/go-openapi/spec
anymore sincek8s.io/kube-openapi
supports all the features required by Kubernetes (kubernetes/kube-openapi#234).Which issue(s) this PR fixes:
Fixes #96303
Does this PR introduce a user-facing change?:
Signed-off-by: Gautier Delorme gautier.delorme@gmail.com