Set DefaulterRemoveUnknownOrOmitableFields webhook option to preserve previous behavior

sbueringer · sbueringer · commit 7720a944eabf · 2025-01-16T15:42:04.000+01:00
Signed-off-by: Stefan Büringer buringerst@vmware.com
diff --git a/bootstrap/kubeadm/internal/webhooks/kubeadmconfig.go b/bootstrap/kubeadm/internal/webhooks/kubeadmconfig.go
@@ -33,7 +33,7 @@ import (
 func (webhook *KubeadmConfig) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&bootstrapv1.KubeadmConfig{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/bootstrap/kubeadm/internal/webhooks/kubeadmconfigtemplate.go b/bootstrap/kubeadm/internal/webhooks/kubeadmconfigtemplate.go
@@ -32,7 +32,7 @@ import (
 func (webhook *KubeadmConfigTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&bootstrapv1.KubeadmConfigTemplate{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/controlplane/kubeadm/internal/webhooks/kubeadm_control_plane.go b/controlplane/kubeadm/internal/webhooks/kubeadm_control_plane.go
@@ -47,7 +47,7 @@ import (
 func (webhook *KubeadmControlPlane) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&controlplanev1.KubeadmControlPlane{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/controlplane/kubeadm/internal/webhooks/kubeadmcontrolplanetemplate.go b/controlplane/kubeadm/internal/webhooks/kubeadmcontrolplanetemplate.go
@@ -36,7 +36,7 @@ import (
 func (webhook *KubeadmControlPlaneTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&controlplanev1.KubeadmControlPlaneTemplate{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/exp/addons/internal/webhooks/clusterresourceset_webhook.go b/exp/addons/internal/webhooks/clusterresourceset_webhook.go
@@ -39,7 +39,7 @@ type ClusterResourceSet struct{}
 func (webhook *ClusterResourceSet) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&addonsv1.ClusterResourceSet{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/exp/internal/webhooks/machinepool.go b/exp/internal/webhooks/machinepool.go
@@ -49,7 +49,7 @@ func (webhook *MachinePool) SetupWebhookWithManager(mgr ctrl.Manager) error {
 
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&expv1.MachinePool{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/cluster.go b/internal/webhooks/cluster.go
@@ -57,7 +57,7 @@ func (webhook *Cluster) SetupWebhookWithManager(mgr ctrl.Manager) error {
 
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.Cluster{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/clusterclass.go b/internal/webhooks/clusterclass.go
@@ -45,7 +45,7 @@ import (
 func (webhook *ClusterClass) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.ClusterClass{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/machine.go b/internal/webhooks/machine.go
@@ -40,7 +40,7 @@ const defaultNodeDeletionTimeout = 10 * time.Second
 func (webhook *Machine) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.Machine{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/machinedeployment.go b/internal/webhooks/machinedeployment.go
@@ -48,7 +48,7 @@ func (webhook *MachineDeployment) SetupWebhookWithManager(mgr ctrl.Manager) erro
 
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.MachineDeployment{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/machinehealthcheck.go b/internal/webhooks/machinehealthcheck.go
@@ -52,7 +52,7 @@ func SetMinNodeStartupTimeout(d metav1.Duration) {
 func (webhook *MachineHealthCheck) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.MachineHealthCheck{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/machineset.go b/internal/webhooks/machineset.go
@@ -49,7 +49,7 @@ func (webhook *MachineSet) SetupWebhookWithManager(mgr ctrl.Manager) error {
 
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&clusterv1.MachineSet{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/internal/webhooks/runtime/extensionconfig_webhook.go b/internal/webhooks/runtime/extensionconfig_webhook.go
@@ -42,7 +42,7 @@ type ExtensionConfig struct{}
 func (webhook *ExtensionConfig) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&runtimev1.ExtensionConfig{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/docker/internal/webhooks/dockercluster_webhook.go b/test/infrastructure/docker/internal/webhooks/dockercluster_webhook.go
@@ -36,7 +36,7 @@ type DockerCluster struct{}
 func (webhook *DockerCluster) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.DockerCluster{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/docker/internal/webhooks/dockerclustertemplate_webhook.go b/test/infrastructure/docker/internal/webhooks/dockerclustertemplate_webhook.go
@@ -38,7 +38,7 @@ type DockerClusterTemplate struct{}
 func (webhook *DockerClusterTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.DockerClusterTemplate{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/inmemory/internal/webhooks/inmemorycluster_webhook.go b/test/infrastructure/inmemory/internal/webhooks/inmemorycluster_webhook.go
@@ -33,7 +33,7 @@ type InMemoryCluster struct{}
 func (webhook *InMemoryCluster) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.InMemoryCluster{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/inmemory/internal/webhooks/inmemoryclustertemplate_webhook.go b/test/infrastructure/inmemory/internal/webhooks/inmemoryclustertemplate_webhook.go
@@ -33,7 +33,7 @@ type InMemoryClusterTemplate struct{}
 func (webhook *InMemoryClusterTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.InMemoryClusterTemplate{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/inmemory/internal/webhooks/inmemorymachine_webhook.go b/test/infrastructure/inmemory/internal/webhooks/inmemorymachine_webhook.go
@@ -33,7 +33,7 @@ type InMemoryMachine struct{}
 func (webhook *InMemoryMachine) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.InMemoryMachine{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }
diff --git a/test/infrastructure/inmemory/internal/webhooks/inmemorymachinetemplate_webhook.go b/test/infrastructure/inmemory/internal/webhooks/inmemorymachinetemplate_webhook.go
@@ -33,7 +33,7 @@ type InMemoryMachineTemplate struct{}
 func (webhook *InMemoryMachineTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {
 	return ctrl.NewWebhookManagedBy(mgr).
 		For(&infrav1.InMemoryMachineTemplate{}).
-		WithDefaulter(webhook).
+		WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).
 		WithValidator(webhook).
 		Complete()
 }

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,7 @@ import (`
`33`	`33`	`func (webhook *KubeadmConfig) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`34`	`34`	`return ctrl.NewWebhookManagedBy(mgr).`
`35`	`35`	`For(&bootstrapv1.KubeadmConfig{}).`
`36`		`- WithDefaulter(webhook).`
	`36`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`37`	`37`	`WithValidator(webhook).`
`38`	`38`	`Complete()`
`39`	`39`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ import (`
`32`	`32`	`func (webhook *KubeadmConfigTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`33`	`33`	`return ctrl.NewWebhookManagedBy(mgr).`
`34`	`34`	`For(&bootstrapv1.KubeadmConfigTemplate{}).`
`35`		`- WithDefaulter(webhook).`
	`35`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`36`	`36`	`WithValidator(webhook).`
`37`	`37`	`Complete()`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,7 @@ import (`
`47`	`47`	`func (webhook *KubeadmControlPlane) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`48`	`48`	`return ctrl.NewWebhookManagedBy(mgr).`
`49`	`49`	`For(&controlplanev1.KubeadmControlPlane{}).`
`50`		`- WithDefaulter(webhook).`
	`50`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`51`	`51`	`WithValidator(webhook).`
`52`	`52`	`Complete()`
`53`	`53`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ import (`
`36`	`36`	`func (webhook *KubeadmControlPlaneTemplate) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`37`	`37`	`return ctrl.NewWebhookManagedBy(mgr).`
`38`	`38`	`For(&controlplanev1.KubeadmControlPlaneTemplate{}).`
`39`		`- WithDefaulter(webhook).`
	`39`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`40`	`40`	`WithValidator(webhook).`
`41`	`41`	`Complete()`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ type ClusterResourceSet struct{}`
`39`	`39`	`func (webhook *ClusterResourceSet) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`40`	`40`	`return ctrl.NewWebhookManagedBy(mgr).`
`41`	`41`	`For(&addonsv1.ClusterResourceSet{}).`
`42`		`- WithDefaulter(webhook).`
	`42`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`43`	`43`	`WithValidator(webhook).`
`44`	`44`	`Complete()`
`45`	`45`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ func (webhook *MachinePool) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`49`	`49`
`50`	`50`	`return ctrl.NewWebhookManagedBy(mgr).`
`51`	`51`	`For(&expv1.MachinePool{}).`
`52`		`- WithDefaulter(webhook).`
	`52`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`53`	`53`	`WithValidator(webhook).`
`54`	`54`	`Complete()`
`55`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ func (webhook *Cluster) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`57`	`57`
`58`	`58`	`return ctrl.NewWebhookManagedBy(mgr).`
`59`	`59`	`For(&clusterv1.Cluster{}).`
`60`		`- WithDefaulter(webhook).`
	`60`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`61`	`61`	`WithValidator(webhook).`
`62`	`62`	`Complete()`
`63`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ import (`
`45`	`45`	`func (webhook *ClusterClass) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`46`	`46`	`return ctrl.NewWebhookManagedBy(mgr).`
`47`	`47`	`For(&clusterv1.ClusterClass{}).`
`48`		`- WithDefaulter(webhook).`
	`48`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`49`	`49`	`WithValidator(webhook).`
`50`	`50`	`Complete()`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@ const defaultNodeDeletionTimeout = 10 * time.Second`
`40`	`40`	`func (webhook *Machine) SetupWebhookWithManager(mgr ctrl.Manager) error {`
`41`	`41`	`return ctrl.NewWebhookManagedBy(mgr).`
`42`	`42`	`For(&clusterv1.Machine{}).`
`43`		`- WithDefaulter(webhook).`
	`43`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`44`	`44`	`WithValidator(webhook).`
`45`	`45`	`Complete()`
`46`	`46`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ func (webhook *MachineDeployment) SetupWebhookWithManager(mgr ctrl.Manager) erro`
`48`	`48`
`49`	`49`	`return ctrl.NewWebhookManagedBy(mgr).`
`50`	`50`	`For(&clusterv1.MachineDeployment{}).`
`51`		`- WithDefaulter(webhook).`
	`51`	`+ WithDefaulter(webhook, admission.DefaulterRemoveUnknownOrOmitableFields).`
`52`	`52`	`WithValidator(webhook).`
`53`	`53`	`Complete()`
`54`	`54`	`}`