Merge pull request #993 from ykulazhenkov/pr-fix-main-exit-code

SchSeba · web-flow · commit 8627a03fe48b · 2025-12-24T19:20:47.000+02:00
fix: exit with non-zero code on error in config daemon
diff --git a/cmd/sriov-network-config-daemon/main.go b/cmd/sriov-network-config-daemon/main.go
@@ -17,6 +17,7 @@ package main
 
 import (
 	"flag"
+	"os"
 
 	"github.com/spf13/cobra"
 	"sigs.k8s.io/controller-runtime/pkg/log"
@@ -44,5 +45,6 @@ func init() {
 func main() {
 	if err := rootCmd.Execute(); err != nil {
 		log.Log.Error(err, "error executing sriov-network-config-daemon")
+		os.Exit(1)
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ package main`
`17`	`17`
`18`	`18`	`import (`
`19`	`19`	`"flag"`
	`20`	`+ "os"`
`20`	`21`
`21`	`22`	`"github.com/spf13/cobra"`
`22`	`23`	`"sigs.k8s.io/controller-runtime/pkg/log"`
`@@ -44,5 +45,6 @@ func init() {`
`44`	`45`	`func main() {`
`45`	`46`	`if err := rootCmd.Execute(); err != nil {`
`46`	`47`	`log.Log.Error(err, "error executing sriov-network-config-daemon")`
	`48`	`+ os.Exit(1)`
`47`	`49`	`}`
`48`	`50`	`}`