From a1c1ef31df6f492ad26e1412d7f3b924c2d3aee5 Mon Sep 17 00:00:00 2001 From: Dan Winship Date: Sat, 20 Apr 2024 10:00:20 -0400 Subject: [PATCH] Remove unnecessary e2e feature.UDP [Feature:UDP] was probably added in the past by analogy to [Feature:SCTP], but is unnecessary since UDP support has always been required. --- test/e2e/feature/feature.go | 3 --- test/e2e/network/netpol/network_policy.go | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/test/e2e/feature/feature.go b/test/e2e/feature/feature.go index db0200bbe42..4477ca157f9 100644 --- a/test/e2e/feature/feature.go +++ b/test/e2e/feature/feature.go @@ -327,9 +327,6 @@ var ( // TODO: document the feature (owning SIG, when to use this feature for a test) TopologyManager = framework.WithFeature(framework.ValidFeatures.Add("TopologyManager")) - // TODO: document the feature (owning SIG, when to use this feature for a test) - UDP = framework.WithFeature(framework.ValidFeatures.Add("UDP")) - // TODO: document the feature (owning SIG, when to use this feature for a test) Upgrade = framework.WithFeature(framework.ValidFeatures.Add("Upgrade")) diff --git a/test/e2e/network/netpol/network_policy.go b/test/e2e/network/netpol/network_policy.go index f15aa024789..165a8c7c0b8 100644 --- a/test/e2e/network/netpol/network_policy.go +++ b/test/e2e/network/netpol/network_policy.go @@ -893,7 +893,7 @@ var _ = common.SIGDescribe("Netpol", func() { ValidateOrFail(k8s, &TestCase{ToPort: 80, Protocol: v1.ProtocolTCP, Reachability: reachability}) }) - f.It("should enforce ingress policy allowing any port traffic to a server on a specific protocol", feature.NetworkPolicy, feature.UDP, func(ctx context.Context) { + f.It("should enforce ingress policy allowing any port traffic to a server on a specific protocol", feature.NetworkPolicy, func(ctx context.Context) { protocols := []v1.Protocol{protocolTCP, protocolUDP} ports := []int32{80} k8s = initializeResources(ctx, f, protocols, ports)