From 0e84cc939a977aa1f25c24c82cbad02aab4b5784 Mon Sep 17 00:00:00 2001
From: Dan Finneran <dan@thebsdbox.co.uk>
Date: Thu, 18 Jun 2026 15:48:34 +0000
Subject: [PATCH] Moves logging level back to the global setting

Signed-off-by: Dan Finneran <dan@thebsdbox.co.uk>
---
 pkg/bgp/server.go         | 4 ++--
 pkg/manager/manager.go    | 3 +--
 pkg/manager/worker/bgp.go | 2 +-
 3 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/pkg/bgp/server.go b/pkg/bgp/server.go
index b77fadd7..5c64aa50 100644
--- a/pkg/bgp/server.go
+++ b/pkg/bgp/server.go
@@ -24,7 +24,7 @@ type Server struct {
 }
 
 // NewBGPServer takes a configuration and returns a running BGP server instance
-func NewBGPServer(c kubevip.BGPConfig) (b *Server, err error) {
+func NewBGPServer(c kubevip.BGPConfig, logLevel log.Level) (b *Server, err error) {
 	if c.AS == 0 {
 		return nil, fmt.Errorf("you need to provide AS")
 	}
@@ -38,7 +38,7 @@ func NewBGPServer(c kubevip.BGPConfig) (b *Server, err error) {
 	}
 	bgpLogger := log.Default()
 	lvl := &log.LevelVar{}
-	lvl.Set(log.LevelInfo)
+	lvl.Set(logLevel)
 
 	b = &Server{
 		s:       gobgp.NewBgpServer(gobgp.LoggerOption(bgpLogger, lvl)),
diff --git a/pkg/manager/manager.go b/pkg/manager/manager.go
index f8acd70e..06b084b9 100644
--- a/pkg/manager/manager.go
+++ b/pkg/manager/manager.go
@@ -227,8 +227,7 @@ func New(ctx context.Context, configMap string, config *kubevip.Config) (*Manage
 		if err != nil {
 			return nil, err
 		}
-
-		bgpServer, err = bgp.NewBGPServer(config.BGPConfig)
+		bgpServer, err = bgp.NewBGPServer(config.BGPConfig, log.Level(config.Logging))
 		if err != nil {
 			return nil, fmt.Errorf("creating BGP server: %w", err)
 		}
diff --git a/pkg/manager/worker/bgp.go b/pkg/manager/worker/bgp.go
index c709f866..5aea93ec 100644
--- a/pkg/manager/worker/bgp.go
+++ b/pkg/manager/worker/bgp.go
@@ -45,7 +45,7 @@ func NewBGP(arpMgr *arp.Manager, intfMgr *networkinterface.Manager,
 func (b *BGP) Configure(ctx context.Context, _ *sync.WaitGroup) error {
 	var err error
 	if b.bgpServer == nil {
-		b.bgpServer, err = bgp.NewBGPServer(b.config.BGPConfig)
+		b.bgpServer, err = bgp.NewBGPServer(b.config.BGPConfig, log.Level(b.config.Logging))
 		if err != nil {
 			return fmt.Errorf("creating BGP server: %w", err)
 		}