webhook: detect errors correctly on update

Signed-off-by: Tuomas Katila <[email protected]>

Author: tkatila

pkg/apis/deviceplugin/v1/webhook_common.go

@@ -118,10 +118,10 @@ func (r *commonDevicePluginValidator) ValidateCreate(ctx context.Context, obj ru
 }
 
 // ValidateUpdate implements admission.CustomValidator so a webhook will be registered for the type.
-func (r *commonDevicePluginValidator) ValidateUpdate(ctx context.Context, oldObj runtime.Object, newObj runtime.Object) (admission.Warnings, error) {
+func (r *commonDevicePluginValidator) ValidateUpdate(ctx context.Context, _ runtime.Object, newObj runtime.Object) (admission.Warnings, error) {
 	logf.FromContext(ctx).Info("validate update")
 
-	switch v := oldObj.(type) {
+	switch v := newObj.(type) {
 	case *DlbDevicePlugin:
 		return nil, v.validatePlugin(r)
 	case *DsaDevicePlugin:
@@ -137,7 +137,7 @@ func (r *commonDevicePluginValidator) ValidateUpdate(ctx context.Context, oldObj
 	case *SgxDevicePlugin:
 		return nil, v.validatePlugin(r)
 	default:
-		return nil, fmt.Errorf("%w: expected an xDevicePlugin object but got %T", errObjType, oldObj)
+		return nil, fmt.Errorf("%w: expected an xDevicePlugin object but got %T", errObjType, newObj)
 	}
 }