[V1] conf/test_case_checklist:add checklist for special case

Message ID 20210903182734.25439-1-zhiminx.huang@intel.com (mailing list archive)
State Superseded
Headers
Series [V1] conf/test_case_checklist:add checklist for special case |

Commit Message

Huang, ZhiminX Sept. 3, 2021, 6:27 p.m. UTC
  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(+)
  

Comments

Huang, ZhiminX Sept. 3, 2021, 9:57 a.m. UTC | #1
> -----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 >
  
Tu, Lijuan Sept. 6, 2021, 3:40 a.m. UTC | #2
> -----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.
  
Huang, ZhiminX Sept. 6, 2021, 4:29 a.m. UTC | #3
> -----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?
  
Tu, Lijuan Sept. 6, 2021, 7 a.m. UTC | #4
> -----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.
  
Huang, ZhiminX Sept. 6, 2021, 7:17 a.m. UTC | #5
> -----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.
  

Patch

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": [