No-op: Remove unnecessary nil check (#3091)

tbavelier · web-flow · commit e71cfc3c68ea · 2026-06-05T15:37:40.000+02:00
diff --git a/internal/controller/datadogagent/override/podtemplatespec.go b/internal/controller/datadogagent/override/podtemplatespec.go
@@ -104,7 +104,7 @@ func PodTemplateSpec(logger logr.Logger, manager feature.PodTemplateManagers, ov
 		manager.EnvFromVar().AddEnvFromVar(&e)
 	}
 
-	if override.CELWorkloadExclude != nil && len(override.CELWorkloadExclude) > 0 {
+	if len(override.CELWorkloadExclude) > 0 {
 		jsonConfig, err := json.Marshal(override.CELWorkloadExclude)
 		if err != nil {
 			logger.Error(err, "failed to convert to JSON")

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ func PodTemplateSpec(logger logr.Logger, manager feature.PodTemplateManagers, ov`
`104`	`104`	`manager.EnvFromVar().AddEnvFromVar(&e)`
`105`	`105`	`}`
`106`	`106`
`107`		`- if override.CELWorkloadExclude != nil && len(override.CELWorkloadExclude) > 0 {`
	`107`	`+ if len(override.CELWorkloadExclude) > 0 {`
`108`	`108`	`jsonConfig, err := json.Marshal(override.CELWorkloadExclude)`
`109`	`109`	`if err != nil {`
`110`	`110`	`logger.Error(err, "failed to convert to JSON")`