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
prevent negotation on connections that dont' require it #38903
prevent negotation on connections that dont' require it #38903
Conversation
@@ -91,6 +91,9 @@ type ClientAccessFactory interface { | |||
RESTClient() (*restclient.RESTClient, error) | |||
// Returns a client.Config for accessing the Kubernetes server. | |||
ClientConfig() (*restclient.Config, error) | |||
// BareClientConfig returns a client.Config that has NOT been negotiated. It's | |||
// just directions to the server. People use this to build RESTMappers on top of |
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.
directions?
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.
directions?
connection info? address, port, certs, etc
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.
nvm, was thinking this should have been "direct" but I get it now.
LGTM |
@k8s-bot test this [submit-queue is verifying that this PR is safe to merge] |
Automatic merge from submit-queue (batch tested with PRs 38080, 38903) |
Not all clientconfigs require negotiation. Before there was a patch (hardcoded to an instance variable) that allowed this. This updates the code to allow the same as before.
@kubernetes/sig-cli @fabianofranz
@ncdc you'll want to pick