fix(pruner): ensure enforcedConfigLevel is always defaulted and DeepCopy is correct

  The SetDefaults change in commit 774867c guarded GlobalConfig.SetDefaults()
  behind a nil check to allow users to set historyLimit to null without it
  resetting. However, this meant enforcedConfigLevel was never filled in when
  a user provided a partial global-config (e.g. only ttlSecondsAfterFinished).
  The pruner controller then defaulted to the "resource" level internally,
  which silently ignores all namespace-level selector rules.

  Additionally, DeepCopyInto for TektonPrunerConfig copied the *GlobalConfig
  pointer instead of the underlying struct. This caused the Knative defaulting
  webhook to see no difference between the original and the copy after
  SetDefaults ran, so the patch was always null and defaults were never applied.

Signed-off-by: Shubham Bhardwaj <shubbhar@redhat.com>

Author: infernus01

pkg/apis/operator/v1alpha1/tektonpruner_defaults.go

@@ -35,5 +35,8 @@ func (p *Pruner) SetDefaults() {
 	if p.GlobalConfig == nil {
 		p.GlobalConfig = &config.GlobalConfig{}
 		p.GlobalConfig.SetDefaults()
+	} else if p.GlobalConfig.EnforcedConfigLevel == nil {
+		v := config.EnforcedConfigLevelGlobal
+		p.GlobalConfig.EnforcedConfigLevel = &v
 	}
 }

pkg/apis/operator/v1alpha1/tektonpruner_types.go

@@ -111,5 +111,8 @@ func (p *Pruner) IsDisabled() bool {
 
 func (in *TektonPrunerConfig) DeepCopyInto(out *TektonPrunerConfig) {
 	*out = *in
-	return
+	if in.GlobalConfig != nil {
+		out.GlobalConfig = new(config.GlobalConfig)
+		*out.GlobalConfig = *in.GlobalConfig
+	}
 }