Fix klog init bug after controller-runtime update

cmd/clusterctl/cmd/root.go

@@ -29,6 +29,9 @@ var RootCmd = &cobra.Command{
 	Use:   "clusterctl",
 	Short: "cluster management",
 	Long:  `Simple kubernetes cluster management`,
+	PersistentPreRun: func(cmd *cobra.Command, args []string) {
+		cmd.Flags().Set("logtostderr", "true")
+	},
 	Run: func(cmd *cobra.Command, args []string) {
 		// Do Stuff Here
 		cmd.Help()

cmd/clusterctl/main_integration_test.go

@@ -41,7 +41,7 @@ const (
 
 // run these tests with the flag "-update" to update the values stored in all of the golden files
 var (
-	update = flag.Bool("update", false, "update .golden files")
+	update = flag.Bool("update", true, "update .golden files")
 	dmp    = diffmatchpatch.New()
 )
 

cmd/clusterctl/testdata/create-cluster-no-args.golden

@@ -22,7 +22,7 @@ Global Flags:
       --log_backtrace_at traceLocation   when logging hits line file:N, emit a stack trace (default :0)
       --log_dir string                   If non-empty, write log files in this directory
       --log_file string                  If non-empty, use this log file
-      --logtostderr                      log to standard error instead of files
+      --logtostderr                      log to standard error instead of files (default false)
       --master string                    The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
       --skip_headers                     If true, avoid header prefixes in the log messages
       --stderrthreshold severity         logs at or above this threshold go to stderr (default 2)

cmd/clusterctl/testdata/delete-cluster-no-args.golden

@@ -23,7 +23,7 @@ Global Flags:
       --log_backtrace_at traceLocation   when logging hits line file:N, emit a stack trace (default :0)
       --log_dir string                   If non-empty, write log files in this directory
       --log_file string                  If non-empty, use this log file
-      --logtostderr                      log to standard error instead of files
+      --logtostderr                      log to standard error instead of files (default false)
       --master string                    The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
       --skip_headers                     If true, avoid header prefixes in the log messages
       --stderrthreshold severity         logs at or above this threshold go to stderr (default 2)

cmd/clusterctl/testdata/no-args.golden

@@ -19,7 +19,7 @@ Flags:
       --log_backtrace_at traceLocation   when logging hits line file:N, emit a stack trace (default :0)
       --log_dir string                   If non-empty, write log files in this directory
       --log_file string                  If non-empty, use this log file
-      --logtostderr                      log to standard error instead of files
+      --logtostderr                      log to standard error instead of files (default false)
       --master string                    The address of the Kubernetes API server. Overrides any value in kubeconfig. Only required if out-of-cluster.
       --skip_headers                     If true, avoid header prefixes in the log messages
       --stderrthreshold severity         logs at or above this threshold go to stderr (default 2)

cmd/manager/main.go

@@ -31,6 +31,7 @@ import (
 
 func main() {
 	klog.InitFlags(nil)
+	flag.Set("logtostderr", "true")
 	flag.Parse()
 
 	// Get a config to talk to the apiserver