Fix incorrect usage of logrus when formatting string is present

This fix tries to fix logrus formatting by adding `f` to the end of `logrus.[Error|Warn|Debug|Fatal|Panic|Info](` when formatting string is present but the function `logrus.[Error|Warn|Debug|Fatal|Panic|Info](` is used (incorrectly). This fix is related to #23459, and is a follow up of #23461. Signed-off-by: Yong Tang <[email protected]>
alexmavr · Jun 11, 2016 · d917723 · d917723
1 parent 96110f3
commit d917723
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/daemon/daemon_unix.go b/daemon/daemon_unix.go
@@ -362,7 +362,7 @@ func verifyContainerResources(resources *containertypes.Resources, sysInfo *sysi
 	}
 	if resources.CPUPercent > 0 {
 		warnings = append(warnings, "%s does not support CPU percent. Percent discarded.", runtime.GOOS)
-		logrus.Warn("%s does not support CPU percent. Percent discarded.", runtime.GOOS)
+		logrus.Warnf("%s does not support CPU percent. Percent discarded.", runtime.GOOS)
 		resources.CPUPercent = 0
 	}
 

diff --git a/integration-cli/events_utils.go b/integration-cli/events_utils.go
@@ -89,7 +89,7 @@ func (e *eventObserver) Match(match eventMatcher, process eventMatchProcessor) {
 		err = io.EOF
 	}
 
-	logrus.Debug("EventObserver scanner loop finished: %v", err)
+	logrus.Debugf("EventObserver scanner loop finished: %v", err)
 	e.disconnectionError = err
 }