-
Notifications
You must be signed in to change notification settings - Fork 0
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
cmd/server/kube #42
cmd/server/kube #42
Conversation
@@ -26,7 +25,7 @@ func TestKubeletDefaults(t *testing.T) { | |||
// If the default changes (new fields are added, or default values change), we want to know | |||
// Once we've reacted to the changes appropriately in BuildKubernetesNodeConfig(), update this expected default to match the new upstream defaults | |||
expectedDefaults := &kubeletoptions.KubeletServer{ | |||
AuthPath: flag.NewStringFlag(""), | |||
//AuthPath: flag.NewStringFlag(""), |
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 is gone?
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.
It's gone, kube commit 2f6027a
) | ||
|
||
func NewSourceAPIOpenShift( |
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.
Do you plan to merge that somehow with the upstream one?
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.
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.
Yeah, ideally we'll just use what's upstream, but it's not available yet.
39ae150
to
47eb8e3
Compare
No description provided.