Use fmt.printf() to end with a newline

update pull request

cmd/kubeadm/app/cmd/phases/workflow/doc_test.go

@@ -50,7 +50,7 @@ func ExamplePhase() {
 			}
 
 			// implement your phase logic...
-			fmt.Printf("%v", d.Data())
+			fmt.Printf("%v\n", d.Data())
 			return nil
 		},
 	}
@@ -67,7 +67,7 @@ func ExamplePhase() {
 			}
 
 			// implement your phase logic...
-			fmt.Printf("%v", d.Data())
+			fmt.Printf("%v\n", d.Data())
 			return nil
 		},
 	}
@@ -91,7 +91,7 @@ func ExampleRunner_Run() {
 			}
 
 			// implement your phase logic...
-			fmt.Printf("%v", d.Data())
+			fmt.Printf("%v\n", d.Data())
 			return nil
 		},
 	}

cmd/kubeadm/app/phases/upgrade/postupgrade.go

@@ -174,7 +174,7 @@ func BackupAPIServerCertIfNeeded(cfg *kubeadmapi.InitConfiguration, dryRun bool)
 	// Don't fail the upgrade phase if failing to backup kube-apiserver cert and key, just continue rotating the cert
 	// TODO: We might want to reconsider this choice.
 	if err := backupAPIServerCertAndKey(certAndKeyDir); err != nil {
-		fmt.Printf("[postupgrade] WARNING: failed to backup kube-apiserver cert and key: %v", err)
+		fmt.Printf("[postupgrade] WARNING: failed to backup kube-apiserver cert and key: %v\n", err)
 	}
 	return certsphase.CreateCertAndKeyFilesWithCA(
 		&certsphase.KubeadmCertAPIServer,

cmd/kubeadm/app/util/arguments.go

@@ -63,7 +63,7 @@ func ParseArgumentListToMap(arguments []string) map[string]string {
 		// Warn in all other cases, but don't error out. This can happen only if the user has edited the argument list by hand, so they might know what they are doing
 		if err != nil {
 			if i != 0 {
-				fmt.Printf("[kubeadm] WARNING: The component argument %q could not be parsed correctly. The argument must be of the form %q. Skipping...", arg, "--")
+				fmt.Printf("[kubeadm] WARNING: The component argument %q could not be parsed correctly. The argument must be of the form %q. Skipping...\n", arg, "--")
 			}
 			continue
 		}