Skip to content
This repository has been archived by the owner on Sep 12, 2023. It is now read-only.

Don't import k8s.io/kubernetes #110

Closed
gaocegege opened this issue Feb 1, 2021 · 3 comments · Fixed by #111
Closed

Don't import k8s.io/kubernetes #110

gaocegege opened this issue Feb 1, 2021 · 3 comments · Fixed by #111

Comments

@gaocegege
Copy link
Member

Ref kubernetes/kubernetes#90358

Ref argoproj/argo-cd#5173
@gaocegege gaocegege mentioned this issue Feb 1, 2021
Merged
@gaocegege gaocegege reopened this Feb 9, 2021
@Jeffwan
Copy link
Member

Jeffwan commented Mar 25, 2021

I thought this has been done? @gaocegege Any left over stories?

@gaocegege
Copy link
Member Author

It is blocked by volcano release. Thus it is still open.

@Jeffwan
Copy link
Member

Jeffwan commented Mar 31, 2021

#111
#123
#127

k8s/kubernetes and volcano.sh/volcano dependencies have been removed. We only need to import api and apimachinery now.

@Jeffwan Jeffwan closed this as completed Mar 31, 2021
georgkaleido pushed a commit to georgkaleido/common that referenced this issue Jun 9, 2022
@otherguy
Fix CI pipeline (kubeflow#110)
3018574 
* Remove kubeval from Danger
* Remove kubeval from CircleCI
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: Remove k8s.io/kubernetes from go mod gaocegege/common
2 participants
@Jeffwan @gaocegege