[V1] tests/cvl_dcf_switch_filter_gtpu: modify the execution order of sub cases

Message ID 1630650226-212245-1-git-send-email-songx.jiale@intel.com (mailing list archive)
State Accepted
Headers
Series [V1] tests/cvl_dcf_switch_filter_gtpu: modify the execution order of sub cases |

Checks

Context Check Description
ci/Intel-dts-test success Testing OK

Commit Message

Jiale, SongX Sept. 3, 2021, 6:23 a.m. UTC
  Because creating rules of the same type will cause rule creation fail, 
adjusting the order of the sub cases can avoid this problem

Signed-off-by: Jiale Song <songx.jiale@intel.com>
---
 tests/TestSuite_cvl_dcf_switch_filter_gtpu.py | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)
  

Comments

Tu, Lijuan Sept. 29, 2021, 3:34 a.m. UTC | #1
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Jiale Song
> Sent: 2021年9月3日 14:24
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: [dts] [PATCH V1] tests/cvl_dcf_switch_filter_gtpu: modify the execution
> order of sub cases
> 
> Because creating rules of the same type will cause rule creation fail, adjusting
> the order of the sub cases can avoid this problem
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>

Applied, thanks
  

Patch

diff --git a/tests/TestSuite_cvl_dcf_switch_filter_gtpu.py b/tests/TestSuite_cvl_dcf_switch_filter_gtpu.py
index 052e97b..830c996 100644
--- a/tests/TestSuite_cvl_dcf_switch_filter_gtpu.py
+++ b/tests/TestSuite_cvl_dcf_switch_filter_gtpu.py
@@ -691,15 +691,15 @@  sv_mac_ipv4_gtpu_eh_ipv4_udp_1 = [
     tv_mac_ipv4_gtpu_eh_ipv4_udp_teid_without_mask,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_qfi,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_l3dst,
-    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3src,
-    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3src_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_udp_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3dst_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3dst_l4src,
 ]
 
 sv_mac_ipv4_gtpu_eh_ipv4_udp_2 = [
-    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3dst_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3src,
+    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3src_l4dst,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_l3src_l4src,
-    tv_mac_ipv4_gtpu_eh_ipv4_udp_l3dst_l4src,
-    tv_mac_ipv4_gtpu_eh_ipv4_udp_l4dst,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_l4src,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_l4src_l4dst,
     tv_mac_ipv4_gtpu_eh_ipv4_udp_teid_l4src_l4dst,
@@ -711,6 +711,7 @@  sv_mac_ipv4_gtpu_eh_ipv4_udp_2 = [
     tv_mac_ipv4_gtpu_eh_ipv4_udp_all,
 ]
 
+
 sv_mac_ipv4_gtpu_eh_ipv4_udp = [sv_mac_ipv4_gtpu_eh_ipv4_udp_1, sv_mac_ipv4_gtpu_eh_ipv4_udp_2]
 
 sv_mac_ipv4_gtpu_eh_ipv6_udp_1 = [eval(str(sv).replace('tv_mac_ipv4_gtpu_eh_ipv4', 'tv_mac_ipv4_gtpu_eh_ipv6')
@@ -1062,15 +1063,15 @@  sv_mac_ipv4_gtpu_eh_ipv4_tcp_1 = [
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_teid_without_mask,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_qfi,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3dst,
-    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3src,
-    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3src_l4src,
+    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3dst_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3dst_l4src,
 ]
 
 sv_mac_ipv4_gtpu_eh_ipv4_tcp_2 = [
+    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3src,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3src_l4dst,
-    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3dst_l4src,
-    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3dst_l4dst,
-    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l4dst,
+    tv_mac_ipv4_gtpu_eh_ipv4_tcp_l3src_l4src,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_l4src,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_l4src_l4dst,
     tv_mac_ipv4_gtpu_eh_ipv4_tcp_teid_l4src_l4dst,