Fixes to logging

Signed-off-by: thebsdbox <[email protected]>
dockerpac · Oct 31, 2022 · 8687d28 · 8687d28
1 parent 5330992
commit 8687d28
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/cmd/kube-vip.go b/cmd/kube-vip.go
@@ -218,7 +218,7 @@ var kubeVipManager = &cobra.Command{
 		}
 
 		// Set the logging level for all subsequent functions
-		log.SetLevel(log.Level(logLevel))
+		log.SetLevel(log.Level(initConfig.Logging))
 
 		// start prometheus server
 		if initConfig.PrometheusHTTPServer != "" {

diff --git a/pkg/kubevip/config_environment.go b/pkg/kubevip/config_environment.go
@@ -6,7 +6,6 @@ import (
 
 	"github.com/kube-vip/kube-vip/pkg/bgp"
 	"github.com/kube-vip/kube-vip/pkg/detector"
-	log "github.com/sirupsen/logrus"
 )
 
 // ParseEnvironment - will popultate the configuration from environment variables
@@ -19,7 +18,7 @@ func ParseEnvironment(c *Config) error {
 		if err != nil {
 			panic("Unable to parse environment variable [vip_loglevel], should be int")
 		}
-		log.SetLevel(log.Level(logLevel))
+		c.Logging = int(logLevel)
 	}
 
 	// Find interface

diff --git a/pkg/kubevip/config_types.go b/pkg/kubevip/config_types.go
@@ -9,6 +9,9 @@ import (
 // Config defines all of the settings for the Kube-Vip Pod
 type Config struct {
 
+	// Logging, settings
+	Logging int `yaml:"logging"`
+
 	// EnableARP, will use ARP to advertise the VIP address
 	EnableARP bool `yaml:"enableARP"`