| Message ID | 20210903182734.25439-1-zhiminx.huang@intel.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | [V1] conf/test_case_checklist:add checklist for special case | expand |
> -----Original Message----- > From: Huang, ZhiminX <zhiminx.huang@intel.com> > Sent: Saturday, September 4, 2021 2:28 AM > To: dts@dpdk.org > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > Subject: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special > case > Tested-by: Zhimin Huang <zhiminx.huang@intel.com >
> -----Original Message----- > From: dts <dts-bounces@dpdk.org> On Behalf Of Zhimin Huang > Sent: 2021年9月4日 2:28 > To: dts@dpdk.org > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > Subject: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case > > test case need different cvl with serial number, it's cover by manual test, so add > checklist > > Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com> Test plan and script should be updated coordinately with the special requirements.
> -----Original Message----- > From: Tu, Lijuan <lijuan.tu@intel.com> > Sent: Monday, September 6, 2021 11:40 AM > To: Huang, ZhiminX <zhiminx.huang@intel.com>; dts@dpdk.org > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special > case > > > > -----Original Message----- > > From: dts <dts-bounces@dpdk.org> On Behalf Of Zhimin Huang > > Sent: 2021年9月4日 2:28 > > To: dts@dpdk.org > > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > > Subject: [dts] [PATCH V1] conf/test_case_checklist:add checklist for > > special case > > > > test case need different cvl with serial number, it's cover by > > manual test, so add checklist > > > > Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com> > > Test plan and script should be updated coordinately with the special > requirements. > Test plan have already marked requirement as two cvl nic, so do I need to add anything?
> -----Original Message----- > From: Huang, ZhiminX <zhiminx.huang@intel.com> > Sent: 2021年9月6日 12:30 > To: Tu, Lijuan <lijuan.tu@intel.com>; dts@dpdk.org > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special > case > > > -----Original Message----- > > From: Tu, Lijuan <lijuan.tu@intel.com> > > Sent: Monday, September 6, 2021 11:40 AM > > To: Huang, ZhiminX <zhiminx.huang@intel.com>; dts@dpdk.org > > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist > > for special case > > > > > > > -----Original Message----- > > > From: dts <dts-bounces@dpdk.org> On Behalf Of Zhimin Huang > > > Sent: 2021年9月4日 2:28 > > > To: dts@dpdk.org > > > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > > > Subject: [dts] [PATCH V1] conf/test_case_checklist:add checklist for > > > special case > > > > > > test case need different cvl with serial number, it's cover by > > > manual test, so add checklist > > > > > > Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com> > > > > Test plan and script should be updated coordinately with the special > > requirements. > > > > Test plan have already marked requirement as two cvl nic, so do I need to add > anything? > What's do you mean " with serial number " ? I misunderstood is as 2 different type cards. e.g. E810CQDA2 and E810XXVDA4 is the right configuration. But two E810CQDA2 is the wrong configuration.
> -----Original Message----- > From: Tu, Lijuan <lijuan.tu@intel.com> > Sent: Monday, September 6, 2021 3:01 PM > To: Huang, ZhiminX <zhiminx.huang@intel.com>; dts@dpdk.org > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special > case > > > > > -----Original Message----- > > From: Huang, ZhiminX <zhiminx.huang@intel.com> > > Sent: 2021年9月6日 12:30 > > To: Tu, Lijuan <lijuan.tu@intel.com>; dts@dpdk.org > > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist > > for special case > > > > > -----Original Message----- > > > From: Tu, Lijuan <lijuan.tu@intel.com> > > > Sent: Monday, September 6, 2021 11:40 AM > > > To: Huang, ZhiminX <zhiminx.huang@intel.com>; dts@dpdk.org > > > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > > > Subject: RE: [dts] [PATCH V1] conf/test_case_checklist:add checklist > > > for special case > > > > > > > > > > -----Original Message----- > > > > From: dts <dts-bounces@dpdk.org> On Behalf Of Zhimin Huang > > > > Sent: 2021年9月4日 2:28 > > > > To: dts@dpdk.org > > > > Cc: Huang, ZhiminX <zhiminx.huang@intel.com> > > > > Subject: [dts] [PATCH V1] conf/test_case_checklist:add checklist > > > > for special case > > > > > > > > test case need different cvl with serial number, it's cover by > > > > manual test, so add checklist > > > > > > > > Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com> > > > > > > Test plan and script should be updated coordinately with the special > > > requirements. > > > > > > > Test plan have already marked requirement as two cvl nic, so do I need > > to add anything? > > > > What's do you mean " with serial number " ? I misunderstood is as 2 different > type cards. > e.g. E810CQDA2 and E810XXVDA4 is the right configuration. But two > E810CQDA2 is the wrong configuration. > Your understanding is correct, it's need different type cards, sorry for my confuse description, I will submit v2 patch to add correct descriptions.
diff --git a/conf/test_case_checklist.json b/conf/test_case_checklist.json index 8cd47dd..218599e 100644 --- a/conf/test_case_checklist.json +++ b/conf/test_case_checklist.json @@ -1773,6 +1773,22 @@ "Comments": "the nic not support this case" } ], + "check_specific_package_loading": [ + { + "OS": [ + "ALL" + ], + "NIC": [ + "columbiaville_100g", + "columbiaville_25g" + ], + "Target": [ + "ALL" + ], + "Bug ID": "", + "Comments": "the case needs >=2 CVL NICs with different Serial Numbers" + } + ], "kernel_2pf_2vf_1vm_mac_add_filter": [ { "OS": [
test case need different cvl with serial number, it's cover by manual test, so add checklist Signed-off-by: Zhimin Huang <zhiminx.huang@intel.com> --- conf/test_case_checklist.json | 16 ++++++++++++++++ 1 file changed, 16 insertions(+)