From f91f739ee26c67fab50a3ff0f6db4e175a6dbcb9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Roberto=20Jim=C3=A9nez=20S=C3=A1nchez?= Date: Tue, 1 Apr 2025 12:37:35 +0200 Subject: [PATCH] Remove unused HACK for testing new configuration (#103172) --- pkg/registry/apis/provisioning/test.go | 16 ---------------- 1 file changed, 16 deletions(-) diff --git a/pkg/registry/apis/provisioning/test.go b/pkg/registry/apis/provisioning/test.go index 6b26e8482d6..85c8b3a2bf8 100644 --- a/pkg/registry/apis/provisioning/test.go +++ b/pkg/registry/apis/provisioning/test.go @@ -7,7 +7,6 @@ import ( "reflect" "time" - apierrors "k8s.io/apimachinery/pkg/api/errors" metav1 "k8s.io/apimachinery/pkg/apis/meta/v1" "k8s.io/apimachinery/pkg/runtime" "k8s.io/apiserver/pkg/registry/rest" @@ -67,21 +66,6 @@ func (s *testConnector) Connect(ctx context.Context, name string, opts runtime.O responder.Error(err) return } - - // TODO: Explore how to better support synchronous validation for the UI (and likely remove this hack) - if name != "new" { - repo, err = s.getter.AsRepository(ctx, &cfg) - if err != nil { - responder.Error(err) - return - } - - // Make sure we are OK with the changes - if cfg.Spec.Type != repo.Config().Spec.Type { - responder.Error(apierrors.NewBadRequest("test config must be the same type")) - return - } - } repo = tmp } }