Skip to content

Commit 046e976

Browse files
committed
cap the num of nodes on the noSNAT test and remove slow and NoSNAT tag
run NoSNAT network test between pods without any feature tag
1 parent 1854839 commit 046e976

File tree

2 files changed

+2
-7
lines changed

2 files changed

+2
-7
lines changed

test/e2e/feature/feature.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -254,10 +254,6 @@ var (
254254
// TODO: document the feature (owning SIG, when to use this feature for a test)
255255
NodeOutOfServiceVolumeDetach = framework.WithFeature(framework.ValidFeatures.Add("NodeOutOfServiceVolumeDetach"))
256256

257-
// Owner: sig-network
258-
// Marks a single test that tests pod-to-pod connectivity between every pair of nodes.
259-
NoSNAT = framework.WithFeature(framework.ValidFeatures.Add("NoSNAT"))
260-
261257
// Owner: sig-network
262258
// Marks a single test that tests cluster DNS performance with many services.
263259
PerformanceDNS = framework.WithFeature(framework.ValidFeatures.Add("PerformanceDNS"))

test/e2e/network/no_snat.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ import (
2929
"k8s.io/apimachinery/pkg/util/wait"
3030
admissionapi "k8s.io/pod-security-admission/api"
3131

32-
"k8s.io/kubernetes/test/e2e/feature"
3332
"k8s.io/kubernetes/test/e2e/framework"
3433
e2enode "k8s.io/kubernetes/test/e2e/framework/node"
3534
e2epod "k8s.io/kubernetes/test/e2e/framework/pod"
@@ -64,15 +63,15 @@ var (
6463

6564
// This test verifies that a Pod on each node in a cluster can talk to Pods on every other node without SNAT.
6665
// We use the [Feature:NoSNAT] tag so that most jobs will skip this test by default.
67-
var _ = common.SIGDescribe("NoSNAT", feature.NoSNAT, framework.WithSlow(), func() {
66+
var _ = common.SIGDescribe("NoSNAT", func() {
6867
f := framework.NewDefaultFramework("no-snat-test")
6968
f.NamespacePodSecurityLevel = admissionapi.LevelPrivileged
7069
ginkgo.It("Should be able to send traffic between Pods without SNAT", func(ctx context.Context) {
7170
cs := f.ClientSet
7271
pc := cs.CoreV1().Pods(f.Namespace.Name)
7372

7473
ginkgo.By("creating a test pod on each Node")
75-
nodes, err := e2enode.GetReadySchedulableNodes(ctx, cs)
74+
nodes, err := e2enode.GetBoundedReadySchedulableNodes(ctx, cs, 3)
7675
framework.ExpectNoError(err)
7776
gomega.Expect(nodes.Items).ToNot(gomega.BeEmpty(), "no Nodes in the cluster")
7877

0 commit comments

Comments
 (0)