fix: capture Run() error in metrics controller goroutine (#330)

The goroutine was referencing the outer 'err' variable from
metrics.NewController() instead of capturing the return value
from metricsController.Run(). By the time the goroutine executes,
the outer err is nil (NewController succeeded), so the log line
always printed a nil error — making failures impossible to diagnose.
Fixes #325

Author: abhaygoudannavar

main.go

@@ -278,8 +278,8 @@ func main() {
 	}
 	kruisegameInformerFactory.Start(signal.Done())
 	go func() {
-		if metricsController.Run(signal) != nil {
-			setupLog.Error(err, "unable to setup metrics controller")
+		if runErr := metricsController.Run(signal); runErr != nil {
+			setupLog.Error(runErr, "unable to setup metrics controller")
 			os.Exit(1)
 		}
 	}()