test suite reviews and discussions
 help / color / Atom feed
* [dts] [PATCH V2] test_plans/cvl_dcf_switch_filter_test_plan.rst: Remove actions vf id 0 test case
@ 2020-07-24 14:07 Nannan Lu
  2020-08-12  5:24 ` Tu, Lijuan
  0 siblings, 1 reply; 2+ messages in thread
From: Nannan Lu @ 2020-07-24 14:07 UTC (permalink / raw)
  To: dts; +Cc: Nannan Lu

Remove actions vf id 0 test case, since the datapath has been enabled
for dcf, so dcf can receive packet now.

Signed-off-by: Nannan Lu <nannan.lu@intel.com>
---
 test_plans/cvl_dcf_switch_filter_test_plan.rst | 30 ++++++--------------------
 1 file changed, 7 insertions(+), 23 deletions(-)

diff --git a/test_plans/cvl_dcf_switch_filter_test_plan.rst b/test_plans/cvl_dcf_switch_filter_test_plan.rst
index 9d415f7..72c37d7 100644
--- a/test_plans/cvl_dcf_switch_filter_test_plan.rst
+++ b/test_plans/cvl_dcf_switch_filter_test_plan.rst
@@ -2030,23 +2030,7 @@ Test case: MAC_IPV6_NAT-T-ESP
 Test case: negative cases
 =========================
 
-Subcase 1: actions vf id 0
---------------------------
-
-1. action to vf 0, the rule can be created, but not take effect::
-
-     testpmd> flow create 0 ingress pattern eth dst is 68:05:ca:8d:ed:a8 / ipv4 src is 192.168.0.1 dst is 192.168.0.2 tos is 4 ttl is 3 / tcp src is 25 dst is 23 / end actions vf id 0 / end
-     testpmd> flow list 0
-
-   check the rule exists in the list.
-
-2. send matched packet::
-
-     sendp([Ether(dst="68:05:ca:8d:ed:a8")/IP(src="192.168.0.1",dst="192.168.0.2",tos=4,ttl=3)/TCP(sport=25,dport=23)/("X"*480)], iface="enp27s0f0", count=1)
-
-   check the packet not to port 0, the rule doesn't take effect.
-
-Subcase 2: can not create rule on vf 1
+Subcase 1: can not create rule on vf 1
 --------------------------------------
 
 1. create rule on vf 1::
@@ -2063,7 +2047,7 @@ Subcase 2: can not create rule on vf 1
 
    there is no rule listed.
 
-Subcase 3: unsupported pattern in comms
+Subcase 2: unsupported pattern in comms
 ---------------------------------------
 
 1. create an SCTP rule which is not supported in comms::
@@ -2082,7 +2066,7 @@ Subcase 3: unsupported pattern in comms
 
    check the rule not exists in the list.
 
-Subcase 4: unsupported pattern in os default, but supported in comms
+Subcase 3: unsupported pattern in os default, but supported in comms
 --------------------------------------------------------------------
 
 1. load os default package and launch testpmd as step 3-8 in Prerequisites.
@@ -2105,7 +2089,7 @@ Subcase 4: unsupported pattern in os default, but supported in comms
 
 5. repeat step 2-5 with also not supported pattern MAC_IPV4_PFCP_NODE/MAC_IPV4_L2TPv3/MAC_IPV4_ESP, get the same result.
 
-Subcase 5: unsupported input set
+Subcase 4: unsupported input set
 --------------------------------
 
 1. create an nvgre rule with unsupported input set field [inner tos]::
@@ -2124,7 +2108,7 @@ Subcase 5: unsupported input set
 
    check the rule not exists in the list.
 
-Subcase 6: invalid vf id
+Subcase 5: invalid vf id
 ------------------------
 
 1. create a rule with invalid vf id 5::
@@ -2143,7 +2127,7 @@ Subcase 6: invalid vf id
 
    check the rule not exists in the list.
 
-Subcase 7: delete a non-existing rule
+Subcase 6: delete a non-existing rule
 -------------------------------------
 
 1. check the rule list::
@@ -2164,7 +2148,7 @@ Subcase 7: delete a non-existing rule
 
    check no error reports.
 
-Subcase 8: add long switch rule
+Subcase 7: add long switch rule
 -------------------------------
 
 Description: A recipe has 5 words, one of which is reserved for switch ID,
-- 
2.7.4


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

* Re: [dts] [PATCH V2] test_plans/cvl_dcf_switch_filter_test_plan.rst: Remove actions vf id 0 test case
  2020-07-24 14:07 [dts] [PATCH V2] test_plans/cvl_dcf_switch_filter_test_plan.rst: Remove actions vf id 0 test case Nannan Lu
@ 2020-08-12  5:24 ` Tu, Lijuan
  0 siblings, 0 replies; 2+ messages in thread
From: Tu, Lijuan @ 2020-08-12  5:24 UTC (permalink / raw)
  To: Lu, Nannan, dts; +Cc: Lu, Nannan

> Remove actions vf id 0 test case, since the datapath has been enabled for dcf, so
> dcf can receive packet now.
> 
> Signed-off-by: Nannan Lu <nannan.lu@intel.com>
> ---
>  test_plans/cvl_dcf_switch_filter_test_plan.rst | 30 ++++++--------------------

Applied, thanks

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

end of thread, back to index

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-24 14:07 [dts] [PATCH V2] test_plans/cvl_dcf_switch_filter_test_plan.rst: Remove actions vf id 0 test case Nannan Lu
2020-08-12  5:24 ` Tu, Lijuan

test suite reviews and discussions

Archives are clonable:
	git clone --mirror http://inbox.dpdk.org/dts/0 dts/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 dts dts/ http://inbox.dpdk.org/dts \
		dts@dpdk.org
	public-inbox-index dts


Newsgroup available over NNTP:
	nntp://inbox.dpdk.org/inbox.dpdk.dts


AGPL code for this site: git clone https://public-inbox.org/ public-inbox