From 63962c7212586d1218a8602b80927e10176a4f1c Mon Sep 17 00:00:00 2001 From: tosinski Date: Mon, 7 Feb 2022 17:24:19 -0800 Subject: [PATCH 1/2] Use wider port range in app filtering integration test --- test/integration/basic_test.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/test/integration/basic_test.go b/test/integration/basic_test.go index 2a4ad182e..f7f34936b 100644 --- a/test/integration/basic_test.go +++ b/test/integration/basic_test.go @@ -159,8 +159,7 @@ func TestSingleUEAttachAndDetach(t *testing.T) { nbAddress: nodeBAddress, ueAddress: ueAddress, upfN3Address: upfN3Address, - // TODO: use wider port range once multi port ranges are supported - sdfFilter: "permit out udp from 192.168.1.1/32 to assigned 80-80", + sdfFilter: "permit out udp from 192.168.1.1/32 to assigned 80-400", ulTEID: 15, dlTEID: 16, sessQFI: 0x09, @@ -172,7 +171,7 @@ func TestSingleUEAttachAndDetach(t *testing.T) { appIP: net.ParseIP("192.168.1.1"), appPrefixLen: 32, appPort: portRange{ - 80, 80, + 80, 400, }, }, // FIXME: there is a dependency on previous test because pfcpiface doesn't clear application IDs properly From 590f84183cfb3064e9495bece68c3baa3637abb4 Mon Sep 17 00:00:00 2001 From: Github Actions Date: Tue, 8 Feb 2022 01:48:13 +0000 Subject: [PATCH 2/2] Actions: Updated with changes from CI --- test/integration/basic_test.go | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/test/integration/basic_test.go b/test/integration/basic_test.go index f7f34936b..72598e696 100644 --- a/test/integration/basic_test.go +++ b/test/integration/basic_test.go @@ -159,11 +159,11 @@ func TestSingleUEAttachAndDetach(t *testing.T) { nbAddress: nodeBAddress, ueAddress: ueAddress, upfN3Address: upfN3Address, - sdfFilter: "permit out udp from 192.168.1.1/32 to assigned 80-400", - ulTEID: 15, - dlTEID: 16, - sessQFI: 0x09, - appQFI: 0x08, + sdfFilter: "permit out udp from 192.168.1.1/32 to assigned 80-400", + ulTEID: 15, + dlTEID: 16, + sessQFI: 0x09, + appQFI: 0x08, }, expected: p4RtValues{ appFilter: appFilter{