Change BindKubeletClientConfigFlags to use - in flag names instead of _

The only user of this function is using pflags and does have name normalization. It might break something out of tree, but that's impossible to prevent...

Change BindKubeletClientConfigFlags to use - in flag names instead of _
3547920d · Eric Paris · 9fdd7935 · 3547920d
Commit 3547920d authored Aug 11, 2015 by Eric Paris
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 6 deletions

flags.go pkg/client/flags.go +6 -6

No files found.
--- a/pkg/client/flags.go
+++ b/pkg/client/flags.go
@@ -45,10 +45,10 @@ func BindClientConfigFlags(flags FlagSet, config *Config) {
 }
 func BindKubeletClientConfigFlags(flags FlagSet, config *KubeletConfig) {
-	flags.BoolVar(&config.EnableHttps, "kubelet_https", config.EnableHttps, "Use https for kubelet connections")
+	flags.BoolVar(&config.EnableHttps, "kubelet-https", config.EnableHttps, "Use https for kubelet connections")
-	flags.UintVar(&config.Port, "kubelet_port", config.Port, "Kubelet port")
+	flags.UintVar(&config.Port, "kubelet-port", config.Port, "Kubelet port")
-	flags.DurationVar(&config.HTTPTimeout, "kubelet_timeout", config.HTTPTimeout, "Timeout for kubelet operations")
+	flags.DurationVar(&config.HTTPTimeout, "kubelet-timeout", config.HTTPTimeout, "Timeout for kubelet operations")
-	flags.StringVar(&config.CertFile, "kubelet_client_certificate", config.CertFile, "Path to a client key file for TLS.")
+	flags.StringVar(&config.CertFile, "kubelet-client-certificate", config.CertFile, "Path to a client key file for TLS.")
-	flags.StringVar(&config.KeyFile, "kubelet_client_key", config.KeyFile, "Path to a client key file for TLS.")
+	flags.StringVar(&config.KeyFile, "kubelet-client-key", config.KeyFile, "Path to a client key file for TLS.")
-	flags.StringVar(&config.CAFile, "kubelet_certificate_authority", config.CAFile, "Path to a cert. file for the certificate authority.")
+	flags.StringVar(&config.CAFile, "kubelet-certificate-authority", config.CAFile, "Path to a cert. file for the certificate authority.")
 }