diff mbox series

[V2] tests/rteflow_priority: update case code for dpdk code change

Message ID 1634199023-65745-1-git-send-email-songx.jiale@intel.com (mailing list archive)
State Accepted
Headers show
Series [V2] tests/rteflow_priority: update case code for dpdk code change | expand

Checks

Context Check Description
ci/Intel-dts-suite-test success Testing OK
ci/Intel-dts-doc-test fail Testing issues

Commit Message

Jiale Song Oct. 14, 2021, 8:10 a.m. UTC
when dpdk fixed a bug, the code design changed, so in order to test the fdir rule,
need to add "mark" to the rule

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

Comments

Zhimin Huang Oct. 21, 2021, 2:02 a.m. UTC | #1
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Jiale Song
> Sent: Thursday, October 14, 2021 4:10 PM
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: [dts] [PATCH V2] tests/rteflow_priority: update case code for dpdk
> code change
> 
> when dpdk fixed a bug, the code design changed, so in order to test the fdir rule,
> need to add "mark" to the rule
> 
> Signed-off-by: Jiale Song <songx.jiale@intel.com>
> ---
Add dpdk code changed commit: e4a0a7599d974f05665fec3e4c251659f0b11453
Tu, Lijuan Oct. 26, 2021, 6:14 a.m. UTC | #2
> -----Original Message-----
> From: dts <dts-bounces@dpdk.org> On Behalf Of Huang, ZhiminX
> Sent: 2021年10月21日 10:02
> To: dts@dpdk.org
> Cc: Jiale, SongX <songx.jiale@intel.com>
> Subject: Re: [dts] [PATCH V2] tests/rteflow_priority: update case code for dpdk
> code change
> 
> > -----Original Message-----
> > From: dts <dts-bounces@dpdk.org> On Behalf Of Jiale Song
> > Sent: Thursday, October 14, 2021 4:10 PM
> > To: dts@dpdk.org
> > Cc: Jiale, SongX <songx.jiale@intel.com>
> > Subject: [dts] [PATCH V2] tests/rteflow_priority: update case code for
> > dpdk code change
> >
> > when dpdk fixed a bug, the code design changed, so in order to test
> > the fdir rule, need to add "mark" to the rule
> >
> > Signed-off-by: Jiale Song <songx.jiale@intel.com>
> > ---
> Add dpdk code changed commit:
> e4a0a7599d974f05665fec3e4c251659f0b11453

Applied
diff mbox series

Patch

diff --git a/test_plans/rteflow_priority_test_plan.rst b/test_plans/rteflow_priority_test_plan.rst
index b1815ce..4ca1c22 100644
--- a/test_plans/rteflow_priority_test_plan.rst
+++ b/test_plans/rteflow_priority_test_plan.rst
@@ -77,14 +77,14 @@  Patterns in this case:
 
 #. Create a rule with priority 0, Check the flow can be created but it will map to fdir filter::
 
-    testpmd> flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end
+    testpmd> flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end
     ice_interrupt_handler(): OICR: MDD event
     ice_flow_create(): Succeeded to create (1) flow
     Flow rule #0 created
 
 #. Create a rule with priority 1, check the flow can not be created for the vallue of priority is 0 in non-pipeline mode::
 
-    testpmd> flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end
+    testpmd> flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end
     ice_flow_create(): Failed to create flow
     Caught error type 4 (priority field): cause: 0x7ffe24e65738, Not support priority.: Invalid argument
 
diff --git a/tests/TestSuite_rteflow_priority.py b/tests/TestSuite_rteflow_priority.py
index 23aea82..61607c1 100755
--- a/tests/TestSuite_rteflow_priority.py
+++ b/tests/TestSuite_rteflow_priority.py
@@ -204,9 +204,9 @@  class TestRteflowPriority(TestCase):
         out = self.dut.send_expect(command, "testpmd> ", 120)
         self.logger.debug(out)
         
-        out=self.dut.send_expect("flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end", "testpmd>", 15)       
+        out=self.dut.send_expect("flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end", "testpmd>", 15)
         self.verify( "Successed" and "(1)" in out, "failed: rule can't be created to fdir")
-        out=self.dut.send_expect("flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end", "testpmd>", 15)       
+        out=self.dut.send_expect("flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end", "testpmd>", 15)
         self.verify( "Failed" in out, "failed: default value of priority is 0 in non-pipeline mode")
         self.dut.send_expect("flow flush 0", "testpmd>", 20)
         self.dut.send_expect("quit", "#", 50)
@@ -216,9 +216,9 @@  class TestRteflowPriority(TestCase):
         out = self.dut.send_expect(command, "testpmd> ", 120)
         self.logger.debug(out)
         
-        out=self.dut.send_expect("flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end", "testpmd>", 15)       
+        out=self.dut.send_expect("flow create 0 priority 0 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end", "testpmd>", 15)
         self.verify( "Successed" and "(1)" in out, "failed: rule can't be created to fdir")
-        out=self.dut.send_expect("flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / end", "testpmd>", 15)       
+        out=self.dut.send_expect("flow create 0 priority 1 ingress pattern eth / ipv4 src is 192.168.0.2 dst is 192.168.0.3 tos is 4 / end actions queue index 2 / mark / end", "testpmd>", 15)
         self.verify( "Failed" in out, "failed: default value of priority is 0 in non-pipeline mode")
         self.dut.send_expect("flow flush 0", "testpmd>", 20)
         self.dut.send_expect("quit", "#", 50)