Merge pull request #53719 from dixudx/fix_azure_disk_format_log_error

Automatic merge from submit-queue (batch tested with PRs 53872, 53719). If you want to cherry-pick this change to another branch, please follow the instructions <a href="https://github.com/kubernetes/community/blob/master/contributors/devel/cherry-picks.md">here</a>. fix azureDisk warns about disk format failure **What this PR does / why we need it**: **Which issue this PR fixes** *(optional, in `fixes #<issue number>(, fixes #<issue_number>, ...)` format, will close that issue when PR gets merged)*: fixes #53713 **Special notes for your reviewer**: **Release note**: ```release-note None ```

Merge pull request #53719 from dixudx/fix_azure_disk_format_log_error
6265d217 · Kubernetes Submit Queue · GitHub · 6b8e769c · f2f88204 · 6265d217
Commit 6265d217 authored Oct 18, 2017 by Kubernetes Submit Queue Committed by GitHub Oct 18, 2017
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

azure_common_linux.go pkg/volume/azure_dd/azure_common_linux.go +3 -2

No files found.
--- a/pkg/volume/azure_dd/azure_common_linux.go
+++ b/pkg/volume/azure_dd/azure_common_linux.go
@@ -150,9 +150,10 @@ func formatIfNotFormatted(disk string, fstype string, exec mount.Exec) {
 		_, err := exec.Run("mkfs."+fstype, args...)
 		if err == nil {
 			// the disk has been formatted successfully try to mount it again.
-			glog.Infof("azureDisk - Disk successfully formatted (mkfs): %s - %s %s", fstype, disk, "tt")
+			glog.Infof("azureDisk - Disk successfully formatted with 'mkfs.%s %v'", fstype, args)
+		} else {
+			glog.Warningf("azureDisk - Error formatting volume with 'mkfs.%s %v': %v", fstype, args, err)
 		}
-		glog.Warningf("azureDisk - format of disk %q failed: type:(%q) target:(%q) options:(%q)error:(%v)", disk, fstype, "tt", "o", err)
 	} else {
 		if err != nil {
 			glog.Warningf("azureDisk - Failed to check if the disk %s formatted with error %s, will attach anyway", disk, err)