test suite reviews and discussions
 help / color / mirror / Atom feed
From: "Xu, Ke1" <ke1.xu@intel.com>
To: "dts@dpdk.org" <dts@dpdk.org>
Cc: "Tu, Lijuan" <lijuan.tu@intel.com>,
	"Peng, Yuan" <yuan.peng@intel.com>, "Fu, Qi" <qi.fu@intel.com>,
	"Huang, ZhiminX" <zhiminx.huang@intel.com>,
	"Li, HongboX" <hongbox.li@intel.com>
Subject: RE: [DTS][PATCH V2 0/4] update existing cases and add new sub-cases to validate port representor action and represented port action
Date: Wed, 21 Dec 2022 10:38:05 +0000	[thread overview]
Message-ID: <PH7PR11MB605633E426AB37CE75962F8CD3EB9@PH7PR11MB6056.namprd11.prod.outlook.com> (raw)
In-Reply-To: <20221221101252.59230-1-ke1.xu@intel.com>


> -----Original Message-----
> From: Xu, Ke1 <ke1.xu@intel.com>
> Sent: Wednesday, December 21, 2022 6:13 PM
> To: dts@dpdk.org
> Cc: Tu, Lijuan <lijuan.tu@intel.com>; Peng, Yuan <yuan.peng@intel.com>; Fu,
> Qi <qi.fu@intel.com>; Xu, Ke1 <ke1.xu@intel.com>
> Subject: [DTS][PATCH V2 0/4] update existing cases and add new sub-cases
> to validate port representor action and represented port action
> 
> Deprecated TO_VF action contains a 'vf original' action which is not covered
> in  previous tests. This patch applys equivilant 'vf original' rule to previous
> cases for 'vf id' action to cover 'vf original' action.
> 
> This patch adds the following subcases to cover the negative scenario of 'port
> representor' action and 'represented port' action replacing 'vf original' action
> and 'vf id' action.
> 
> 1. Add second PF with VFs for following cases 2. Add a subcase to validate
> creating 'represented port' action on a representor 3. Add a subcase to
> validate creating 'represented port' action on a VF 4. Add a subcase to
> validate creating 'represented port' action to a VF 5. Add two subcases to
> validate creating 'represented port' action to DCFs 6. Add a subcase to
> validate creating 'represented port' action to a representor of another DCF 7.
> Add a subcase to validate creating 'port representor' action on a representor
> 8. Add a subcase to validate creating 'port representor' action on a VF 9. Add
> a subcase to validate creating 'port representor' action to a VF 10. Add a
> subcase to validate creating 'port representor' action to a representor 11.
> Add a subcase to validate creating 'port representor' action to another DCF
> 12. Add a subcase to validate deprecated action 13. Add a subcase to validate
> using an invalid port id

No different modification from V1 patch. Only commit messages changed.

According to V1 patch record:
https://inbox.dpdk.org/dts/PH8PR11MB71202D1CCB1BD539F504DF849FE09@PH8PR11MB7120.namprd11.prod.outlook.com/

Tested-by: Hongbo Li < hongbox.li@intel.com>

> 
> Ke Xu (4):
>   test_plans/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action
>   test_plans/ice_dcf_switch_filter: update existing TO_VF action cases
>     to cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing TO_VF action cases to
>     cover port representor action from not covered vf original action
>   tests/ice_dcf_switch_filter: update existing negative cases to
>     validate represented port action and port representor action
> 
>  .../ice_dcf_switch_filter_test_plan.rst       | 921 ++++++++++++++----
>  tests/TestSuite_ice_dcf_switch_filter.py      | 692 ++++++++-----
>  2 files changed, 1209 insertions(+), 404 deletions(-)
> 
> --
> 2.25.1


  parent reply	other threads:[~2022-12-21 10:53 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-12-21 10:12 Ke Xu
2022-12-21 10:12 ` [DTS][PATCH V2 1/4] test_plans/ice_dcf_switch_filter: update existing negative cases to validate represented port action and port representor action Ke Xu
2022-12-21 10:12 ` [DTS][PATCH V2 2/4] test_plans/ice_dcf_switch_filter: update existing TO_VF action cases to cover port representor action from not covered vf original action Ke Xu
2022-12-21 10:12 ` [DTS][PATCH V2 3/4] tests/ice_dcf_switch_filter: " Ke Xu
2022-12-21 10:12 ` [DTS][PATCH V2 4/4] tests/ice_dcf_switch_filter: update existing negative cases to validate represented port action and port representor action Ke Xu
2022-12-22  4:07   ` Peng, Yuan
2022-12-21 10:38 ` Xu, Ke1 [this message]
2022-12-22  7:45 ` [DTS][PATCH V2 0/4] update existing cases and add new sub-cases to validate port representor action and represented port action Tu, Lijuan

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=PH7PR11MB605633E426AB37CE75962F8CD3EB9@PH7PR11MB6056.namprd11.prod.outlook.com \
    --to=ke1.xu@intel.com \
    --cc=dts@dpdk.org \
    --cc=hongbox.li@intel.com \
    --cc=lijuan.tu@intel.com \
    --cc=qi.fu@intel.com \
    --cc=yuan.peng@intel.com \
    --cc=zhiminx.huang@intel.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).