-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Validate api version in kubectl --patch and --override #4819
Conversation
22bf9d7
to
4bd5543
Compare
We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm. |
4bd5543
to
e7ab977
Compare
CLAs look good, thanks! |
LGTM. Will merge once CI passes. |
|
Too be expected. This would break ./hack/test-cmd.sh until #4793 is merged, but may be fixed in master. I'll rebase. |
a026002
to
e644fce
Compare
This is good to go. |
Rerunning travis since it's been a few days since the last run. Will merge when it comes back green. |
e644fce
to
25fac68
Compare
Validate api version in kubectl --patch and --override
InterfacesFor() returns an error when the apiVersion is not found where CodecFor does not. This breaks ./hack/test-cmd.sh until #4793 is merged. Also related to #4771.