Remove deprecated spec.preserveUnknownFields in favor of x-kubernetes-preserve-unknown-fields

meskinemohamed · meskinemohamed · commit 96e39cd96586 · 2025-12-01T21:15:46.000+01:00
- Remove config/crd/patches/preserve_unknown_fields_false.yaml patch file - Remove patch reference from config/crd/kustomization.yaml - Update kubebuilder annotations from +kubebuilder:validation:XPreserveUnknownFields to +kubebuilder:pruning:PreserveUnknownFields per KubeBuilder docs - Regenerate manifests Fixes #3996 Signed-off-by: meskinemohamed <mohamed.meskine@dataiku.com>
diff --git a/apis/connection/v1alpha1/connection_types.go b/apis/connection/v1alpha1/connection_types.go
@@ -31,7 +31,7 @@ type ConnectionSpec struct {
 	Driver string `json:"driver"`
 	// +kubebuilder:validation:Required
 	// +kubebuilder:validation:Schemaless
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Config *types.Anything `json:"config"`
 }
 
diff --git a/apis/mutations/unversioned/modifyset_types.go b/apis/mutations/unversioned/modifyset_types.go
@@ -60,7 +60,7 @@ type ModifySetParameters struct {
 	// Values describes the values provided to the operation as `values.fromList`.
 	// +kubebuilder:validation:Schemaless
 	// +kubebuilder:validation:Type=object
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Values Values `json:"values,omitempty"`
 }
 
diff --git a/apis/mutations/unversioned/value.go b/apis/mutations/unversioned/value.go
@@ -31,7 +31,7 @@ var validFields = map[Field]bool{
 type AssignField struct {
 	// Value is a constant value that will be assigned to `location`
 	// +kubebuilder:validation:Schemaless
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Value *types.Anything `json:"value,omitempty"`
 
 	// FromMetadata assigns a value from the specified metadata field.
diff --git a/apis/mutations/v1/modifyset_types.go b/apis/mutations/v1/modifyset_types.go
@@ -60,7 +60,7 @@ type ModifySetParameters struct {
 	// Values describes the values provided to the operation as `values.fromList`.
 	// +kubebuilder:validation:Schemaless
 	// +kubebuilder:validation:Type=object
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Values Values `json:"values,omitempty"`
 }
 
diff --git a/apis/mutations/v1/value.go b/apis/mutations/v1/value.go
@@ -8,7 +8,7 @@ import (
 type AssignField struct {
 	// Value is a constant value that will be assigned to `location`
 	// +kubebuilder:validation:Schemaless
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Value *types.Anything `json:"value,omitempty"`
 
 	// FromMetadata assigns a value from the specified metadata field.
diff --git a/apis/mutations/v1alpha1/modifyset_types.go b/apis/mutations/v1alpha1/modifyset_types.go
@@ -60,7 +60,7 @@ type ModifySetParameters struct {
 	// Values describes the values provided to the operation as `values.fromList`.
 	// +kubebuilder:validation:Schemaless
 	// +kubebuilder:validation:Type=object
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Values Values `json:"values,omitempty"`
 }
 
diff --git a/apis/mutations/v1alpha1/value.go b/apis/mutations/v1alpha1/value.go
@@ -8,7 +8,7 @@ import (
 type AssignField struct {
 	// Value is a constant value that will be assigned to `location`
 	// +kubebuilder:validation:Schemaless
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Value *types.Anything `json:"value,omitempty"`
 
 	// FromMetadata assigns a value from the specified metadata field.
diff --git a/apis/mutations/v1beta1/modifyset_types.go b/apis/mutations/v1beta1/modifyset_types.go
@@ -60,7 +60,7 @@ type ModifySetParameters struct {
 	// Values describes the values provided to the operation as `values.fromList`.
 	// +kubebuilder:validation:Schemaless
 	// +kubebuilder:validation:Type=object
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Values Values `json:"values,omitempty"`
 }
 
diff --git a/apis/mutations/v1beta1/value.go b/apis/mutations/v1beta1/value.go
@@ -8,7 +8,7 @@ import (
 type AssignField struct {
 	// Value is a constant value that will be assigned to `location`
 	// +kubebuilder:validation:Schemaless
-	// +kubebuilder:validation:XPreserveUnknownFields
+	// +kubebuilder:pruning:PreserveUnknownFields
 	Value *types.Anything `json:"value,omitempty"`
 
 	// FromMetadata assigns a value from the specified metadata field.
diff --git a/config/crd/kustomization.yaml b/config/crd/kustomization.yaml
@@ -22,13 +22,6 @@ resources:
 bases:
 - ../../vendor/github.com/open-policy-agent/frameworks/constraint/deploy
 
-patches:
-- target:
-    group: apiextensions.k8s.io
-    version: v1
-    kind: CustomResourceDefinition
-  path: patches/preserve_unknown_fields_false.yaml
-
 patchesJson6902:
 - target:
     group: apiextensions.k8s.io
diff --git a/config/crd/patches/preserve_unknown_fields_false.yaml b/config/crd/patches/preserve_unknown_fields_false.yaml
diff --git a/manifest_staging/charts/gatekeeper/crds/assign-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/assign-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: AssignList
     plural: assign
     singular: assign
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
diff --git a/manifest_staging/charts/gatekeeper/crds/assignimage-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/assignimage-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: AssignImageList
     plural: assignimage
     singular: assignimage
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/charts/gatekeeper/crds/assignmetadata-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/assignmetadata-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: AssignMetadataList
     plural: assignmetadata
     singular: assignmetadata
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
diff --git a/manifest_staging/charts/gatekeeper/crds/config-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/config-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConfigList
     plural: configs
     singular: config
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/charts/gatekeeper/crds/configpodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/configpodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConfigPodStatusList
     plural: configpodstatuses
     singular: configpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/connection-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/connection-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConnectionList
     plural: connections
     singular: connection
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/charts/gatekeeper/crds/connectionpodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/connectionpodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConnectionPodStatusList
     plural: connectionpodstatuses
     singular: connectionpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/charts/gatekeeper/crds/constraintpodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/constraintpodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConstraintPodStatusList
     plural: constraintpodstatuses
     singular: constraintpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/constrainttemplatepodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/constrainttemplatepodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ConstraintTemplatePodStatusList
     plural: constrainttemplatepodstatuses
     singular: constrainttemplatepodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/expansiontemplate-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/expansiontemplate-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ExpansionTemplateList
     plural: expansiontemplate
     singular: expansiontemplate
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/charts/gatekeeper/crds/expansiontemplatepodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/expansiontemplatepodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ExpansionTemplatePodStatusList
     plural: expansiontemplatepodstatuses
     singular: expansiontemplatepodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/modifyset-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/modifyset-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ModifySetList
     plural: modifyset
     singular: modifyset
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
diff --git a/manifest_staging/charts/gatekeeper/crds/mutatorpodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/mutatorpodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: MutatorPodStatusList
     plural: mutatorpodstatuses
     singular: mutatorpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/provider-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/provider-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ProviderList
     plural: providers
     singular: provider
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - deprecated: true
diff --git a/manifest_staging/charts/gatekeeper/crds/providerpodstatus-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/providerpodstatus-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: ProviderPodStatusList
     plural: providerpodstatuses
     singular: providerpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
diff --git a/manifest_staging/charts/gatekeeper/crds/syncset-customresourcedefinition.yaml b/manifest_staging/charts/gatekeeper/crds/syncset-customresourcedefinition.yaml
@@ -14,7 +14,6 @@ spec:
     listKind: SyncSetList
     plural: syncsets
     singular: syncset
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
diff --git a/manifest_staging/deploy/gatekeeper.yaml b/manifest_staging/deploy/gatekeeper.yaml
@@ -45,7 +45,6 @@ spec:
     listKind: AssignList
     plural: assign
     singular: assign
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
@@ -1130,7 +1129,6 @@ spec:
     listKind: AssignImageList
     plural: assignimage
     singular: assignimage
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
@@ -1473,7 +1471,6 @@ spec:
     listKind: AssignMetadataList
     plural: assignmetadata
     singular: assignmetadata
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
@@ -2408,7 +2405,6 @@ spec:
     listKind: ConfigPodStatusList
     plural: configpodstatuses
     singular: configpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -2480,7 +2476,6 @@ spec:
     listKind: ConfigList
     plural: configs
     singular: config
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
@@ -2632,7 +2627,6 @@ spec:
     listKind: ConnectionPodStatusList
     plural: connectionpodstatuses
     singular: connectionpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
@@ -2711,7 +2705,6 @@ spec:
     listKind: ConnectionList
     plural: connections
     singular: connection
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1alpha1
@@ -2810,7 +2803,6 @@ spec:
     listKind: ConstraintPodStatusList
     plural: constraintpodstatuses
     singular: constraintpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -2908,7 +2900,6 @@ spec:
     listKind: ConstraintTemplatePodStatusList
     plural: constrainttemplatepodstatuses
     singular: constrainttemplatepodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -3428,7 +3419,6 @@ spec:
     listKind: ExpansionTemplateList
     plural: expansiontemplate
     singular: expansiontemplate
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1
@@ -3681,7 +3671,6 @@ spec:
     listKind: ExpansionTemplatePodStatusList
     plural: expansiontemplatepodstatuses
     singular: expansiontemplatepodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -3756,7 +3745,6 @@ spec:
     listKind: ModifySetList
     plural: modifyset
     singular: modifyset
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1
@@ -4748,7 +4736,6 @@ spec:
     listKind: MutatorPodStatusList
     plural: mutatorpodstatuses
     singular: mutatorpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -4828,7 +4815,6 @@ spec:
     listKind: ProviderPodStatusList
     plural: providerpodstatuses
     singular: providerpodstatus
-  preserveUnknownFields: false
   scope: Namespaced
   versions:
   - name: v1beta1
@@ -4922,7 +4908,6 @@ spec:
     listKind: ProviderList
     plural: providers
     singular: provider
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - deprecated: true
@@ -5135,7 +5120,6 @@ spec:
     listKind: SyncSetList
     plural: syncsets
     singular: syncset
-  preserveUnknownFields: false
   scope: Cluster
   versions:
   - name: v1alpha1

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ type ConnectionSpec struct {`
`31`	`31`	Driver string `json:"driver"`
`32`	`32`	`// +kubebuilder:validation:Required`
`33`	`33`	`// +kubebuilder:validation:Schemaless`
`34`		`- // +kubebuilder:validation:XPreserveUnknownFields`
	`34`	`+ // +kubebuilder:pruning:PreserveUnknownFields`
`35`	`35`	Config *types.Anything `json:"config"`
`36`	`36`	`}`
`37`	`37`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ type ModifySetParameters struct {`
`60`	`60`	// Values describes the values provided to the operation as `values.fromList`.
`61`	`61`	`// +kubebuilder:validation:Schemaless`
`62`	`62`	`// +kubebuilder:validation:Type=object`
`63`		`- // +kubebuilder:validation:XPreserveUnknownFields`
	`63`	`+ // +kubebuilder:pruning:PreserveUnknownFields`
`64`	`64`	Values Values `json:"values,omitempty"`
`65`	`65`	`}`
`66`	`66`