Merge pull request #3066 from 08volt/logging-logs

k8s-ci-robot · web-flow · commit 65b45ff1edb1 · 2026-04-09T14:27:36.000+05:30
feat(logging): add detailed logging for L4 backend service
diff --git a/pkg/l4/backends/backends.go b/pkg/l4/backends/backends.go
@@ -288,6 +288,7 @@ func (p *Pool) EnsureL4BackendService(params L4BackendServiceParams, beLogger kl
 	}
 
 	if params.LogConfigControlEnabled {
+		beLogger.V(2).Info("EnsureL4BackendService: using provided LogConfig for backend service", "logConfig", params.LogConfig)
 		expectedBS.LogConfig = params.LogConfig
 	} else if currentBS != nil {
 		expectedBS.LogConfig = currentBS.LogConfig
diff --git a/pkg/l4/resources/l4.go b/pkg/l4/resources/l4.go
@@ -557,6 +557,7 @@ func (l4 *L4) EnsureInternalLoadBalancer(nodeNames []string, svc *corev1.Service
 		l4.svcLogger.Error(err, "Failed to determine L4 logging config")
 		l4.recorder.Eventf(l4.Service, corev1.EventTypeWarning, l4lbconfig.GetReasonForError(err), "Failed to apply L4LBConfig: %v", err)
 	}
+	l4.svcLogger.V(2).Info("Determined L4 logging config", "logConfig", logConfig, "loggingCondition", loggingCondition)
 	logConfigControlEnabled := loggingCondition.Status == metav1.ConditionTrue
 	result.MetricsState.LoggingControlEnabled = logConfigControlEnabled
 
diff --git a/pkg/l4/resources/l4netlb.go b/pkg/l4/resources/l4netlb.go
@@ -412,6 +412,7 @@ func (l4netlb *L4NetLB) provideBackendService(syncResult *L4NetLBSyncResult, hcL
 		l4netlb.svcLogger.Error(err, "Failed to determine L4 logging config")
 		l4netlb.recorder.Eventf(l4netlb.Service, corev1.EventTypeWarning, l4lbconfig.GetReasonForError(err), "L4LBConfig could not be retrieved: %v", err)
 	}
+	l4netlb.svcLogger.V(2).Info("Determined L4 logging config", "logConfig", logConfig, "loggingCondition", loggingCondition)
 	logConfigControlEnabled := loggingCondition.Status == metav1.ConditionTrue
 	syncResult.MetricsState.LoggingControlEnabled = logConfigControlEnabled
 

Original file line number	Diff line number	Diff line change
`@@ -288,6 +288,7 @@ func (p *Pool) EnsureL4BackendService(params L4BackendServiceParams, beLogger kl`
`288`	`288`	`}`
`289`	`289`
`290`	`290`	`if params.LogConfigControlEnabled {`
	`291`	`+ beLogger.V(2).Info("EnsureL4BackendService: using provided LogConfig for backend service", "logConfig", params.LogConfig)`
`291`	`292`	`expectedBS.LogConfig = params.LogConfig`
`292`	`293`	`} else if currentBS != nil {`
`293`	`294`	`expectedBS.LogConfig = currentBS.LogConfig`
Original file line number	Diff line number	Diff line change
`@@ -557,6 +557,7 @@ func (l4 L4) EnsureInternalLoadBalancer(nodeNames []string, svc corev1.Service`
`557`	`557`	`l4.svcLogger.Error(err, "Failed to determine L4 logging config")`
`558`	`558`	`l4.recorder.Eventf(l4.Service, corev1.EventTypeWarning, l4lbconfig.GetReasonForError(err), "Failed to apply L4LBConfig: %v", err)`
`559`	`559`	`}`
	`560`	`+ l4.svcLogger.V(2).Info("Determined L4 logging config", "logConfig", logConfig, "loggingCondition", loggingCondition)`
`560`	`561`	`logConfigControlEnabled := loggingCondition.Status == metav1.ConditionTrue`
`561`	`562`	`result.MetricsState.LoggingControlEnabled = logConfigControlEnabled`
`562`	`563`
Original file line number	Diff line number	Diff line change
`@@ -412,6 +412,7 @@ func (l4netlb L4NetLB) provideBackendService(syncResult L4NetLBSyncResult, hcL`
`412`	`412`	`l4netlb.svcLogger.Error(err, "Failed to determine L4 logging config")`
`413`	`413`	`l4netlb.recorder.Eventf(l4netlb.Service, corev1.EventTypeWarning, l4lbconfig.GetReasonForError(err), "L4LBConfig could not be retrieved: %v", err)`
`414`	`414`	`}`
	`415`	`+ l4netlb.svcLogger.V(2).Info("Determined L4 logging config", "logConfig", logConfig, "loggingCondition", loggingCondition)`
`415`	`416`	`logConfigControlEnabled := loggingCondition.Status == metav1.ConditionTrue`
`416`	`417`	`syncResult.MetricsState.LoggingControlEnabled = logConfigControlEnabled`
`417`	`418`