Fix webhook interface method validateUpdate args

Author: rajeshvenkata

.github/workflows/golangci-lint.yml

@@ -14,7 +14,7 @@ jobs:
       - uses: actions/checkout@v3
       - uses: actions/setup-go@v3
         with:
-          go-version: '1.22'
+          go-version: '1.23'
           check-latest: true
           cache: true
       - name: golangci-lint

api/v1beta1/etcdadmcluster_webhook.go

@@ -86,7 +86,7 @@ func (r *EtcdadmCluster) ValidateCreate(_ context.Context, obj runtime.Object) (
 }
 
 // ValidateUpdate implements webhook.CustomValidator so a webhook will be registered for the type
-func (r *EtcdadmCluster) ValidateUpdate(_ context.Context, obj, old runtime.Object) (admission.Warnings, error) {
+func (r *EtcdadmCluster) ValidateUpdate(_ context.Context, old, obj runtime.Object) (admission.Warnings, error) {
 	etcdadmCluster, ok := obj.(*EtcdadmCluster)
 	if !ok {
 		return nil, fmt.Errorf("expected an EtcdadmCluster but got %T", obj)
@@ -153,4 +153,4 @@ func (r *EtcdadmCluster) validateCommon() (allErrs field.ErrorList) {
 	}
 
 	return allErrs
-}
+}

api/v1beta1/etcdadmcluster_webhook_test.go

@@ -69,7 +69,7 @@ func TestEtcdadmClusterValidateUpdateCastFail(t *testing.T) {
 	config := &EtcdadmCluster{}
 
 	// Call ValidateUpdate with the wrong type
-	warnings, err := config.ValidateUpdate(context.TODO(), wrongType, &EtcdadmCluster{})
+	warnings, err := config.ValidateUpdate(context.TODO(), &EtcdadmCluster{}, wrongType)
 
 	// Verify that an error is returned
 	g.Expect(warnings).To(BeNil())