From d07b769418d18f06fd5f886490476acfa1432dce Mon Sep 17 00:00:00 2001 From: eun2ce Date: Sat, 27 Sep 2025 21:36:47 +0900 Subject: [PATCH] Replace feature label with WithFeatureGate for KubeletPSI test - Remove feature.KubeletPSI label as it may be redundant for categorization - Add framework.WithFeatureGate(features.KubeletPSI) for proper feature gate checking - Remove redundant utilfeature.DefaultFeatureGate.Enabled check in BeforeEach This addresses the cleanup goal in issue #134172 to replace feature labels with feature gates where appropriate. --- test/e2e_node/summary_test.go | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/test/e2e_node/summary_test.go b/test/e2e_node/summary_test.go index e1b53c6a925..20487f9f678 100644 --- a/test/e2e_node/summary_test.go +++ b/test/e2e_node/summary_test.go @@ -30,7 +30,6 @@ import ( utilfeature "k8s.io/apiserver/pkg/util/feature" kubeletstatsv1alpha1 "k8s.io/kubelet/pkg/apis/stats/v1alpha1" "k8s.io/kubernetes/pkg/features" - "k8s.io/kubernetes/test/e2e/feature" "k8s.io/kubernetes/test/e2e/framework" e2ekubectl "k8s.io/kubernetes/test/e2e/framework/kubectl" e2epod "k8s.io/kubernetes/test/e2e/framework/pod" @@ -368,11 +367,8 @@ var _ = SIGDescribe("Summary API", framework.WithNodeConformance(), func() { }) }) - framework.Context("when querying /stats/summary under pressure", feature.KubeletPSI, framework.WithSerial(), func() { + framework.Context("when querying /stats/summary under pressure", framework.WithSerial(), framework.WithFeatureGate(features.KubeletPSI), func() { ginkgo.BeforeEach(func() { - if !utilfeature.DefaultFeatureGate.Enabled(features.KubeletPSI) { - ginkgo.Skip("KubeletPSI feature gate is not enabled") - } if !IsCgroup2UnifiedMode() { ginkgo.Skip("Skipping since CgroupV2 not used") }