-
Notifications
You must be signed in to change notification settings - Fork 40
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
Unify k8s api endpoints #29
Conversation
LGTM |
}, | ||
NegotiatedSerializer: api.Codecs, | ||
} | ||
thirdPartyResourceConfigInit(&c) |
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.
this and below duplicates the same in dependencies.go. Could it be refactored somehow?
ed74703
to
31a313e
Compare
LGTM. It's a bit weird that the client contains deps and resdefs, both referring to the same restclient, but i can not see better way,at least right now. |
This should be merged after kubernetes/kubernetes#28414 is released |
31a313e
to
c90007c
Compare
The fix will be released in 1.4 afaik. |
kubernetes/kubernetes#28414 is merged now so it will be included in 1.4 release. |
@nebril Can we add there a check for a k8s version inside AC? Then it would crash with useful message. |
c90007c
to
9da8581
Compare
c6070e4
to
453dbdb
Compare
@pigmej done |
453dbdb
to
78566a4
Compare
78566a4
to
a525646
Compare
Reviewed 21 of 21 files at r1, 1 of 1 files at r2. Comments from Reviewable |
fixes #25
This change is