Merge pull request #74 from NVIDIA/log-eventf

adrianchiris · web-flow · commit 26fb2f684ed0 · 2025-01-05T09:11:08.000+02:00
fix bug in the logEventf method
diff --git a/pkg/upgrade/util.go b/pkg/upgrade/util.go
@@ -141,7 +141,7 @@ func GetEventReason() string {
 func logEventf(recorder record.EventRecorder, object runtime.Object, eventType string, reason string, messageFmt string,
 	args ...interface{}) {
 	if recorder != nil {
-		recorder.Eventf(object, eventType, reason, messageFmt, args, nil)
+		recorder.Eventf(object, eventType, reason, messageFmt, args...)
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ func GetEventReason() string {`
`141`	`141`	`func logEventf(recorder record.EventRecorder, object runtime.Object, eventType string, reason string, messageFmt string,`
`142`	`142`	`args ...interface{}) {`
`143`	`143`	`if recorder != nil {`
`144`		`- recorder.Eventf(object, eventType, reason, messageFmt, args, nil)`
	`144`	`+ recorder.Eventf(object, eventType, reason, messageFmt, args...)`
`145`	`145`	`}`
`146`	`146`	`}`
`147`	`147`