[dts] [PATCH V1]framework/dts: fix multi_sections in execution.cfg sharing same dut issue

[dts] [PATCH V1]framework/dts: fix multi_sections in execution.cfg sharing same dut issue

[Xiao Qimai]

*. fix dut of second section in execution.cfg can't compile dpdk

Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
 framework/dts.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/dts.py b/framework/dts.py
index a637209..d6c4fe5 100644
--- a/framework/dts.py
+++ b/framework/dts.py
@@ -557,12 +557,12 @@ def run_all(config_file, pkgName, git, patch, skip_setup,
     stats_report = StatsReporter(output_dir + '/statistics.txt')
     result = Result()
 
-    crbInsts = []
     crbs_conf = CrbsConf()
     crbs = crbs_conf.load_crbs_config()
 
     # for all Execution sections
     for section in config.sections():
+        crbInsts = list()
         dts_parse_param(config, section)
 
         # verify if the delimiter is good if the lists are vertical
-- 
1.8.3.1

Re: [dts] [PATCH V1]framework/dts: fix multi_sections in execution.cfg sharing same dut issue

[Xiao, QimaiX]

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

Tested-by：Xiao Qimai <qimaix.xiao@intel.com>
________________________________
From: Xiao, QimaiX <qimaix.xiao@intel.com>
Sent: Monday, May 11, 2020 2:06 PM
To: dts@dpdk.org <dts@dpdk.org>
Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
Subject: [dts][PATCH V1]framework/dts: fix multi_sections in execution.cfg sharing same dut issue

*. fix dut of second section in execution.cfg can't compile dpdk

Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
---
 framework/dts.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/framework/dts.py b/framework/dts.py
index a637209..d6c4fe5 100644
--- a/framework/dts.py
+++ b/framework/dts.py
@@ -557,12 +557,12 @@ def run_all(config_file, pkgName, git, patch, skip_setup,
     stats_report = StatsReporter(output_dir + '/statistics.txt')
     result = Result()

-    crbInsts = []
     crbs_conf = CrbsConf()
     crbs = crbs_conf.load_crbs_config()

     # for all Execution sections
     for section in config.sections():
+        crbInsts = list()
         dts_parse_param(config, section)

         # verify if the delimiter is good if the lists are vertical
--
1.8.3.1


[-- Attachment #2: Type: text/html, Size: 2969 bytes --]

Re: [dts] [PATCH V1]framework/dts: fix multi_sections in execution.cfg sharing same dut issue

[Tu, Lijuan]

Applied, thanks

> -----Original Message-----
> From: dts [mailto:dts-bounces@dpdk.org] On Behalf Of Xiao Qimai
> Sent: Monday, May 11, 2020 2:07 PM
> To: dts@dpdk.org
> Cc: Xiao, QimaiX <qimaix.xiao@intel.com>
> Subject: [dts] [PATCH V1]framework/dts: fix multi_sections in execution.cfg
> sharing same dut issue
> 
> *. fix dut of second section in execution.cfg can't compile dpdk
> 
> Signed-off-by: Xiao Qimai <qimaix.xiao@intel.com>
> ---
>  framework/dts.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/framework/dts.py b/framework/dts.py index a637209..d6c4fe5
> 100644
> --- a/framework/dts.py
> +++ b/framework/dts.py
> @@ -557,12 +557,12 @@ def run_all(config_file, pkgName, git, patch,
> skip_setup,
>      stats_report = StatsReporter(output_dir + '/statistics.txt')
>      result = Result()
> 
> -    crbInsts = []
>      crbs_conf = CrbsConf()
>      crbs = crbs_conf.load_crbs_config()
> 
>      # for all Execution sections
>      for section in config.sections():
> +        crbInsts = list()
>          dts_parse_param(config, section)
> 
>          # verify if the delimiter is good if the lists are vertical
> --
> 1.8.3.1