Merge pull request #2141 from benluddy/fix-kind-provisioner-log-formatting

openshift-merge-robot · web-flow · commit 8679a21ee189 · 2021-05-05T14:20:11.000-04:00
Fix mishandling of log format strings in Kind test provisioner.
diff --git a/test/e2e/ctx/provisioner_kind.go b/test/e2e/ctx/provisioner_kind.go
@@ -28,34 +28,37 @@ type kindLogAdapter struct {
 	*TestContext
 }
 
-var _ log.Logger = kindLogAdapter{}
+var (
+	_ log.Logger     = kindLogAdapter{}
+	_ log.InfoLogger = kindLogAdapter{}
+)
 
 func (kl kindLogAdapter) Enabled() bool {
 	return true
 }
 
 func (kl kindLogAdapter) Info(message string) {
-	kl.Infof(message)
+	kl.Infof("%s", message)
 }
 
 func (kl kindLogAdapter) Infof(format string, args ...interface{}) {
-	kl.Logf(format, args)
+	kl.Logf(format, args...)
 }
 
 func (kl kindLogAdapter) Warn(message string) {
-	kl.Warnf(message)
+	kl.Warnf("%s", message)
 }
 
 func (kl kindLogAdapter) Warnf(format string, args ...interface{}) {
-	kl.Logf(format, args)
+	kl.Logf(format, args...)
 }
 
 func (kl kindLogAdapter) Error(message string) {
-	kl.Errorf(message)
+	kl.Errorf("%s", message)
 }
 
 func (kl kindLogAdapter) Errorf(format string, args ...interface{}) {
-	kl.Logf(format, args)
+	kl.Logf(format, args...)
 }
 
 func (kl kindLogAdapter) V(log.Level) log.InfoLogger {

Original file line number	Diff line number	Diff line change
`@@ -28,34 +28,37 @@ type kindLogAdapter struct {`
`28`	`28`	`*TestContext`
`29`	`29`	`}`
`30`	`30`
`31`		`-var _ log.Logger = kindLogAdapter{}`
	`31`	`+var (`
	`32`	`+ _ log.Logger = kindLogAdapter{}`
	`33`	`+ _ log.InfoLogger = kindLogAdapter{}`
	`34`	`+)`
`32`	`35`
`33`	`36`	`func (kl kindLogAdapter) Enabled() bool {`
`34`	`37`	`return true`
`35`	`38`	`}`
`36`	`39`
`37`	`40`	`func (kl kindLogAdapter) Info(message string) {`
`38`		`- kl.Infof(message)`
	`41`	`+ kl.Infof("%s", message)`
`39`	`42`	`}`
`40`	`43`
`41`	`44`	`func (kl kindLogAdapter) Infof(format string, args ...interface{}) {`
`42`		`- kl.Logf(format, args)`
	`45`	`+ kl.Logf(format, args...)`
`43`	`46`	`}`
`44`	`47`
`45`	`48`	`func (kl kindLogAdapter) Warn(message string) {`
`46`		`- kl.Warnf(message)`
	`49`	`+ kl.Warnf("%s", message)`
`47`	`50`	`}`
`48`	`51`
`49`	`52`	`func (kl kindLogAdapter) Warnf(format string, args ...interface{}) {`
`50`		`- kl.Logf(format, args)`
	`53`	`+ kl.Logf(format, args...)`
`51`	`54`	`}`
`52`	`55`
`53`	`56`	`func (kl kindLogAdapter) Error(message string) {`
`54`		`- kl.Errorf(message)`
	`57`	`+ kl.Errorf("%s", message)`
`55`	`58`	`}`
`56`	`59`
`57`	`60`	`func (kl kindLogAdapter) Errorf(format string, args ...interface{}) {`
`58`		`- kl.Logf(format, args)`
	`61`	`+ kl.Logf(format, args...)`
`59`	`62`	`}`
`60`	`63`
`61`	`64`	`func (kl kindLogAdapter) V(log.Level) log.InfoLogger {`