test suite reviews and discussions
 help / color / mirror / Atom feed
* [dts] [PATCH V1] tests/ice_limit_value_test: add mark for fdir rule
@ 2022-06-22 17:11 Jiale Song
  2022-06-29  1:38 ` lijuan.tu
  0 siblings, 1 reply; 2+ messages in thread
From: Jiale Song @ 2022-06-22 17:11 UTC (permalink / raw)
  To: dts; +Cc: Jiale Song

1.fdir rule need add mark id
2.synchronize packages in testplan and case

Signed-off-by: Jiale Song <songx.jiale@intel.com>
---
 test_plans/ice_limit_value_test_test_plan.rst |  8 ++++----
 tests/TestSuite_ice_limit_value_test.py       | 14 ++++++++++----
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/test_plans/ice_limit_value_test_test_plan.rst b/test_plans/ice_limit_value_test_test_plan.rst
index ff859038..70311efd 100644
--- a/test_plans/ice_limit_value_test_test_plan.rst
+++ b/test_plans/ice_limit_value_test_test_plan.rst
@@ -350,8 +350,8 @@ Subcase 2: add/delete rules on two VFs
 
 1. create a rule on each vf::
 
-    flow create 0 ingress pattern eth / ipv4 src is 192.168.0.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end
-    flow create 1 ingress pattern eth / ipv4 src is 192.168.0.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end
+    flow create 0 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 1 / end
+    flow create 1 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 2 / end
 
    return the message::
 
@@ -376,8 +376,8 @@ Subcase 2: add/delete rules on two VFs
 
 4. create the rule on each vf one more time, check the rules listed::
 
-    flow create 0 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end
-    flow create 1 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end
+    flow create 0 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 1 / end
+    flow create 1 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 2 / end
 
 5. send matched packet::
 
diff --git a/tests/TestSuite_ice_limit_value_test.py b/tests/TestSuite_ice_limit_value_test.py
index 180c8c6d..b75cea84 100644
--- a/tests/TestSuite_ice_limit_value_test.py
+++ b/tests/TestSuite_ice_limit_value_test.py
@@ -1015,8 +1015,8 @@ class TestICELimitValue(TestCase):
         """
         dut_file_dir = "/tmp/"
         rules = [
-            "flow create 0 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end",
-            "flow create 1 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / end",
+            "flow create 0 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 1 / end",
+            "flow create 1 ingress pattern eth / ipv4 src is 192.168.56.0 dst is 192.1.0.0 tos is 4 / tcp src is 22 dst is 23 / end actions queue index 5 / mark id 2 / end",
         ]
         pkts = [
             'Ether(dst="00:11:22:33:44:55")/IP(src="192.168.56.0",dst="192.1.0.0", tos=4)/TCP(sport=22,dport=23)/Raw("x" * 80)',
@@ -1057,11 +1057,17 @@ class TestICELimitValue(TestCase):
         self.create_fdir_rule(rules, check_stats=True)
         out_0 = self.send_pkts_getouput(pkts=pkts[0], pf_id=0)
         rfc.check_iavf_fdir_mark(
-            out_0, pkt_num=1, check_param={"port_id": 0, "queue": 5}, stats=True
+            out_0,
+            pkt_num=1,
+            check_param={"port_id": 0, "mark_id": 1, "queue": 5},
+            stats=True,
         )
         out_1 = self.send_pkts_getouput(pkts=pkts[1], pf_id=0)
         rfc.check_iavf_fdir_mark(
-            out_1, pkt_num=1, check_param={"port_id": 1, "queue": 5}, stats=True
+            out_1,
+            pkt_num=1,
+            check_param={"port_id": 1, "mark_id": 2, "queue": 5},
+            stats=True,
         )
 
     def launch_testpmd_with_mark(self, rxq=64, txq=64):
-- 
2.17.1


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [dts] [PATCH V1] tests/ice_limit_value_test: add mark for fdir rule
  2022-06-22 17:11 [dts] [PATCH V1] tests/ice_limit_value_test: add mark for fdir rule Jiale Song
@ 2022-06-29  1:38 ` lijuan.tu
  0 siblings, 0 replies; 2+ messages in thread
From: lijuan.tu @ 2022-06-29  1:38 UTC (permalink / raw)
  To: dts, Jiale Song; +Cc: Jiale Song

On Wed, 22 Jun 2022 17:11:14 +0000, Jiale Song <songx.jiale@intel.com> wrote:
> 1.fdir rule need add mark id
> 2.synchronize packages in testplan and case
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>


Applied, thanks

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-06-29  1:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-06-22 17:11 [dts] [PATCH V1] tests/ice_limit_value_test: add mark for fdir rule Jiale Song
2022-06-29  1:38 ` lijuan.tu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).