use apierrors for bad request

Author: AshleyDumaine

bootstrap/api/v1beta1/rke2config_webhook.go

@@ -69,7 +69,7 @@ var _ webhook.CustomDefaulter = &RKE2ConfigCustomDefaulter{}
 func (r *RKE2ConfigCustomDefaulter) Default(_ context.Context, obj runtime.Object) error {
 	rc, ok := obj.(*RKE2Config)
 	if !ok {
-		return fmt.Errorf("expected a RKE2Config object but got %T", obj)
+		return apierrors.NewBadRequest(fmt.Sprintf("expected a RKE2Config but got a %T", obj))
 	}
 
 	DefaultRKE2ConfigSpec(&rc.Spec)

bootstrap/api/v1beta1/rke2configtemplate_webhook.go

@@ -20,6 +20,7 @@ import (
 	"context"
 	"fmt"
 
+	apierrors "k8s.io/apimachinery/pkg/api/errors"
 	"k8s.io/apimachinery/pkg/runtime"
 	ctrl "sigs.k8s.io/controller-runtime"
 	logf "sigs.k8s.io/controller-runtime/pkg/log"
@@ -61,7 +62,7 @@ var _ webhook.CustomDefaulter = &RKE2ConfigTemplateCustomDefaulter{}
 func (r *RKE2ConfigTemplateCustomDefaulter) Default(_ context.Context, obj runtime.Object) error {
 	rct, ok := obj.(*RKE2ConfigTemplate)
 	if !ok {
-		return fmt.Errorf("expected a RKE2ConfigTemplate object but got %T", obj)
+		return apierrors.NewBadRequest(fmt.Sprintf("expected a RKE2ConfigTemplate but got a %T", obj))
 	}
 
 	RKE2configtemplatelog.Info("default", "name", rct.Name)

controlplane/api/v1beta1/rke2controlplane_webhook.go

@@ -66,7 +66,7 @@ var _ webhook.CustomDefaulter = &RKE2ControlPlaneCustomDefaulter{}
 func (rd *RKE2ControlPlaneCustomDefaulter) Default(_ context.Context, obj runtime.Object) error {
 	rcp, ok := obj.(*RKE2ControlPlane)
 	if !ok {
-		return fmt.Errorf("expected a RKE2ControlPlane object but got %T", obj)
+		return apierrors.NewBadRequest(fmt.Sprintf("expected a RKE2ControlPlane but got a %T", obj))
 	}
 
 	bootstrapv1.DefaultRKE2ConfigSpec(&rcp.Spec.RKE2ConfigSpec)

controlplane/api/v1beta1/rke2controlplanetemplate_webhook.go

@@ -63,7 +63,7 @@ var _ webhook.CustomDefaulter = &RKE2ControlPlaneTemplateCustomDefaulter{}
 func (r *RKE2ControlPlaneTemplateCustomDefaulter) Default(_ context.Context, obj runtime.Object) error {
 	rcpt, ok := obj.(*RKE2ControlPlaneTemplate)
 	if !ok {
-		return fmt.Errorf("expected a RKE2ControlPlaneTemplate object but got %T", obj)
+		return apierrors.NewBadRequest(fmt.Sprintf("expected a RKE2ControlPlaneTemplate but got a %T", obj))
 	}
 
 	bootstrapv1.DefaultRKE2ConfigSpec(&rcpt.Spec.Template.Spec.RKE2ConfigSpec)