feat(api): expose log level controls

Fixes #187

Signed-off-by: Sergei Lukianov <[email protected]>

Author: Frostman

api/gateway/v1alpha1/gateway_types.go

@@ -31,6 +31,8 @@ type GatewaySpec struct {
 	Interfaces map[string]GatewayInterface `json:"interfaces,omitempty"`
 	// Neighbors is a list of BGP neighbors
 	Neighbors []GatewayBGPNeighbor `json:"neighbors,omitempty"`
+	// Logs defines the configuration for logging levels
+	Logs GatewayLogs `json:"logs,omitempty"`
 }
 
 // GatewayInterface defines the configuration for a gateway interface
@@ -51,6 +53,32 @@ type GatewayBGPNeighbor struct {
 	ASN uint32 `json:"asn,omitempty"`
 }
 
+// GatewayLogs defines the configuration for logging levels
+type GatewayLogs struct {
+	Default GatewayLogLevel            `json:"default,omitempty"`
+	Tags    map[string]GatewayLogLevel `json:"tags,omitempty"`
+}
+
+type GatewayLogLevel string
+
+const (
+	GatewayLogLevelOff     GatewayLogLevel = "off"
+	GatewayLogLevelError   GatewayLogLevel = "error"
+	GatewayLogLevelWarning GatewayLogLevel = "warning"
+	GatewayLogLevelInfo    GatewayLogLevel = "info"
+	GatewayLogLevelDebug   GatewayLogLevel = "debug"
+	GatewayLogLevelTrace   GatewayLogLevel = "trace"
+)
+
+var GatewayLogLevels = []GatewayLogLevel{
+	GatewayLogLevelOff,
+	GatewayLogLevelError,
+	GatewayLogLevelWarning,
+	GatewayLogLevelInfo,
+	GatewayLogLevelDebug,
+	GatewayLogLevelTrace,
+}
+
 // GatewayStatus defines the observed state of Gateway.
 type GatewayStatus struct{}
 
@@ -83,6 +111,9 @@ func init() {
 }
 
 func (gw *Gateway) Default() {
+	if gw.Spec.Logs.Default == "" {
+		gw.Spec.Logs.Default = GatewayLogLevelInfo
+	}
 }
 
 func (gw *Gateway) Validate(_ context.Context, _ kclient.Reader) error {

api/gateway/v1alpha1/zz_generated.deepcopy.go

@@ -78,6 +78,16 @@ spec:
                   type: object
                 description: Interfaces is a map of interface names to their configurations
                 type: object
+              logs:
+                description: Logs defines the configuration for logging levels
+                properties:
+                  default:
+                    type: string
+                  tags:
+                    additionalProperties:
+                      type: string
+                    type: object
+                type: object
               neighbors:
                 description: Neighbors is a list of BGP neighbors
                 items:

config/crd/bases/gateway.githedgehog.com_gateways.yaml

@@ -97,6 +97,16 @@ spec:
                       type: object
                     description: Interfaces is a map of interface names to their configurations
                     type: object
+                  logs:
+                    description: Logs defines the configuration for logging levels
+                    properties:
+                      default:
+                        type: string
+                      tags:
+                        additionalProperties:
+                          type: string
+                        type: object
+                    type: object
                   neighbors:
                     description: Neighbors is a list of BGP neighbors
                     items:

config/crd/bases/gwint.githedgehog.com_gatewayagents.yaml

@@ -70,6 +70,44 @@ _Appears in:_
 | `mtu` _integer_ | MTU for the interface |  |  |
 
 
+#### GatewayLogLevel
+
+_Underlying type:_ _string_
+
+
+
+
+
+_Appears in:_
+- [GatewayLogs](#gatewaylogs)
+
+| Field | Description |
+| --- | --- |
+| `off` |  |
+| `error` |  |
+| `warning` |  |
+| `info` |  |
+| `debug` |  |
+| `trace` |  |
+
+
+#### GatewayLogs
+
+
+
+GatewayLogs defines the configuration for logging levels
+
+
+
+_Appears in:_
+- [GatewaySpec](#gatewayspec)
+
+| Field | Description | Default | Validation |
+| --- | --- | --- | --- |
+| `default` _[GatewayLogLevel](#gatewayloglevel)_ |  |  |  |
+| `tags` _object (keys:string, values:[GatewayLogLevel](#gatewayloglevel))_ |  |  |  |
+
+
 #### GatewaySpec
 
 
@@ -91,6 +129,7 @@ _Appears in:_
 | `vtepMTU` _integer_ | VTEPMTU is the MTU for the VTEP interface |  |  |
 | `interfaces` _object (keys:string, values:[GatewayInterface](#gatewayinterface))_ | Interfaces is a map of interface names to their configurations |  |  |
 | `neighbors` _[GatewayBGPNeighbor](#gatewaybgpneighbor) array_ | Neighbors is a list of BGP neighbors |  |  |
+| `logs` _[GatewayLogs](#gatewaylogs)_ | Logs defines the configuration for logging levels |  |  |
 
 
 #### GatewayStatus

docs/api.md

@@ -8,6 +8,7 @@ import (
 	"net/netip"
 
 	"go.githedgehog.com/gateway-proto/pkg/dataplane"
+	gwapi "go.githedgehog.com/gateway/api/gateway/v1alpha1"
 	gwintapi "go.githedgehog.com/gateway/api/gwint/v1alpha1"
 )
 
@@ -153,12 +154,23 @@ func buildDataplaneConfig(ag *gwintapi.GatewayAgent) (*dataplane.GatewayConfig,
 		peerings = append(peerings, p)
 	}
 
+	if ag.Spec.Gateway.Logs.Default == "" {
+		ag.Spec.Gateway.Logs.Default = gwapi.GatewayLogLevelInfo
+	}
+	tracing := &dataplane.TracingConfig{
+		Default:  pbLevel(ag.Spec.Gateway.Logs.Default),
+		Taglevel: map[string]dataplane.LogLevel{},
+	}
+	for tag, level := range ag.Spec.Gateway.Logs.Tags {
+		tracing.Taglevel[tag] = pbLevel(level)
+	}
+
 	return &dataplane.GatewayConfig{
 		Generation: ag.Generation,
 		Device: &dataplane.Device{
 			Driver:   dataplane.PacketDriver_KERNEL,
 			Hostname: ag.Name,
-			Loglevel: dataplane.LogLevel_DEBUG,
+			Tracing:  tracing,
 		},
 		Underlay: &dataplane.Underlay{
 			Vrfs: []*dataplane.VRF{
@@ -187,3 +199,22 @@ func buildDataplaneConfig(ag *gwintapi.GatewayAgent) (*dataplane.GatewayConfig,
 		},
 	}, nil
 }
+
+func pbLevel(in gwapi.GatewayLogLevel) dataplane.LogLevel {
+	switch in {
+	case gwapi.GatewayLogLevelOff:
+		return dataplane.LogLevel_OFF
+	case gwapi.GatewayLogLevelError:
+		return dataplane.LogLevel_ERROR
+	case gwapi.GatewayLogLevelWarning:
+		return dataplane.LogLevel_WARNING
+	case gwapi.GatewayLogLevelInfo:
+		return dataplane.LogLevel_INFO
+	case gwapi.GatewayLogLevelDebug:
+		return dataplane.LogLevel_DEBUG
+	case gwapi.GatewayLogLevelTrace:
+		return dataplane.LogLevel_TRACE
+	}
+
+	return dataplane.LogLevel_OFF
+}