test suite reviews and discussions
 help / color / mirror / Atom feed
* Re: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
  2021-09-03 18:27 [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case Zhimin Huang
@ 2021-09-03  9:57 ` Huang, ZhiminX
  2021-09-06  3:40 ` Tu, Lijuan
  1 sibling, 0 replies; 6+ messages in thread
From: Huang, ZhiminX @ 2021-09-03  9:57 UTC (permalink / raw)
  To: dts

[-- Attachment #1: Type: text/plain, Size: 343 bytes --]

> -----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 >



[-- Attachment #2: TestEnable_Package_Download_In_Ice_Driver.log --]
[-- Type: application/octet-stream, Size: 1331 bytes --]

04/09/2021 02:24:57                            dts: 
TEST SUITE : TestEnable_Package_Download_In_Ice_Driver
04/09/2021 02:24:57                            dts: NIC :        columbiaville_100g
04/09/2021 02:24:57             dut.10.240.183.181: 
04/09/2021 02:24:57                         tester: 
04/09/2021 02:24:57                         tester: ifconfig ens7 -promisc
04/09/2021 02:24:57                         tester: 
04/09/2021 02:24:57                         tester: ifconfig ens8 -promisc
04/09/2021 02:24:57                         tester: 
04/09/2021 02:24:57             dut.10.240.183.181: ls /lib/firmware/intel/ice/ddp/ice.pkg
04/09/2021 02:24:57             dut.10.240.183.181: /lib/firmware/intel/ice/ddp/ice.pkg
04/09/2021 02:24:57             dut.10.240.183.181: ls /lib/firmware/updates/intel/ice/ddp/ice.pkg
04/09/2021 02:24:57             dut.10.240.183.181: /lib/firmware/updates/intel/ice/ddp/ice.pkg
04/09/2021 02:24:57             dut.10.240.183.181: \cp /lib/firmware/intel/ice/ddp/ice.pkg /opt/ice.pkg_backup
04/09/2021 02:24:57             dut.10.240.183.181: 
04/09/2021 02:24:57 TestEnable_Package_Download_In_Ice_Driver: Test Case test_check_specific_package_loading Result SKIPPED:
04/09/2021 02:24:57                            dts: 
TEST SUITE ENDED: TestEnable_Package_Download_In_Ice_Driver

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

* [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
@ 2021-09-03 18:27 Zhimin Huang
  2021-09-03  9:57 ` Huang, ZhiminX
  2021-09-06  3:40 ` Tu, Lijuan
  0 siblings, 2 replies; 6+ messages in thread
From: Zhimin Huang @ 2021-09-03 18:27 UTC (permalink / raw)
  To: dts; +Cc: Zhimin Huang

  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(+)

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


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

* Re: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
  2021-09-03 18:27 [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case Zhimin Huang
  2021-09-03  9:57 ` Huang, ZhiminX
@ 2021-09-06  3:40 ` Tu, Lijuan
  2021-09-06  4:29   ` Huang, ZhiminX
  1 sibling, 1 reply; 6+ messages in thread
From: Tu, Lijuan @ 2021-09-06  3:40 UTC (permalink / raw)
  To: Huang, ZhiminX, dts; +Cc: Huang, ZhiminX


> -----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.


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

* Re: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
  2021-09-06  3:40 ` Tu, Lijuan
@ 2021-09-06  4:29   ` Huang, ZhiminX
  2021-09-06  7:00     ` Tu, Lijuan
  0 siblings, 1 reply; 6+ messages in thread
From: Huang, ZhiminX @ 2021-09-06  4:29 UTC (permalink / raw)
  To: Tu, Lijuan, dts

> -----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?


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

* Re: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
  2021-09-06  4:29   ` Huang, ZhiminX
@ 2021-09-06  7:00     ` Tu, Lijuan
  2021-09-06  7:17       ` Huang, ZhiminX
  0 siblings, 1 reply; 6+ messages in thread
From: Tu, Lijuan @ 2021-09-06  7:00 UTC (permalink / raw)
  To: Huang, ZhiminX, dts



> -----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.




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

* Re: [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case
  2021-09-06  7:00     ` Tu, Lijuan
@ 2021-09-06  7:17       ` Huang, ZhiminX
  0 siblings, 0 replies; 6+ messages in thread
From: Huang, ZhiminX @ 2021-09-06  7:17 UTC (permalink / raw)
  To: Tu, Lijuan, dts

> -----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.


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

end of thread, other threads:[~2021-09-06  7:18 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-03 18:27 [dts] [PATCH V1] conf/test_case_checklist:add checklist for special case Zhimin Huang
2021-09-03  9:57 ` Huang, ZhiminX
2021-09-06  3:40 ` Tu, Lijuan
2021-09-06  4:29   ` Huang, ZhiminX
2021-09-06  7:00     ` Tu, Lijuan
2021-09-06  7:17       ` Huang, ZhiminX

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).