From 412626f4024b1acdd856c9047f97d387ee40f884 Mon Sep 17 00:00:00 2001 From: Madhav Jivrajani Date: Thu, 10 Feb 2022 12:54:15 +0530 Subject: [PATCH] resttest: Modify TestContext to have namespace in context Previously, if a request is clusterscoped, the context that was returned has no namespace, ideally the context should contain a metav1.NamespaceNone as the namespace even for cluster scoped requests. Signed-off-by: Madhav Jivrajani --- .../k8s.io/apiserver/pkg/registry/rest/resttest/resttest.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/staging/src/k8s.io/apiserver/pkg/registry/rest/resttest/resttest.go b/staging/src/k8s.io/apiserver/pkg/registry/rest/resttest/resttest.go index c271c66edf6..d5070a41942 100644 --- a/staging/src/k8s.io/apiserver/pkg/registry/rest/resttest/resttest.go +++ b/staging/src/k8s.io/apiserver/pkg/registry/rest/resttest/resttest.go @@ -105,9 +105,6 @@ func (t *Tester) TestNamespace() string { // TestContext returns a namespaced context that will be used when making storage calls. // Namespace is determined by TestNamespace() func (t *Tester) TestContext() context.Context { - if t.clusterScope { - return genericapirequest.NewContext() - } return genericapirequest.WithNamespace(genericapirequest.NewContext(), t.TestNamespace()) }