[dts] [PATCH V1 1/3] add re_run parameter
Xu, HuilongX
huilongx.xu at intel.com
Wed Jun 28 07:38:35 CEST 2017
Hi yong,
Thanks a lot for your comments, I will changed it in next version
> -----Original Message-----
> From: Liu, Yong
> Sent: Saturday, June 24, 2017 12:42 AM
> To: Xu, HuilongX; dts at dpdk.org
> Cc: Xu, HuilongX
> Subject: RE: [dts] [PATCH V1 1/3] add re_run parameter
>
> Huilong,
> Rerun is not supposed to be the default action for failed cases. The default
> value for rerun times should be zero,
>
> > -----Original Message-----
> > From: dts [mailto:dts-bounces at dpdk.org] On Behalf Of xu,huilong
> > Sent: Thursday, June 22, 2017 10:57 PM
> > To: dts at dpdk.org
> > Cc: Xu, HuilongX <huilongx.xu at intel.com>
> > Subject: [dts] [PATCH V1 1/3] add re_run parameter
> >
> > add a new re_run parameter for failed case re run time. the default
> > value is 1.
> >
> > Signed-off-by: xu,huilong <huilongx.xu at intel.com>
> > ---
> > framework/dts.py | 7 +++++--
> > framework/main.py | 5 ++++-
> > 2 files changed, 9 insertions(+), 3 deletions(-)
> >
> > diff --git a/framework/dts.py b/framework/dts.py index
> > 369599d..48be694 100644
> > --- a/framework/dts.py
> > +++ b/framework/dts.py
> > @@ -424,7 +424,7 @@ def dts_run_suite(duts, tester, test_suites, target):
> > def run_all(config_file, pkgName, git, patch, skip_setup,
> > read_cache, project, suite_dir, test_cases,
> > base_dir, output_dir, verbose, virttype, debug,
> > - debugcase, commands):
> > + debugcase, re_run, commands):
> > """
> > Main process of DTS, it will run all test suites in the config file.
> > """
> > @@ -465,6 +465,9 @@ def run_all(config_file, pkgName, git, patch,
> > skip_setup,
> > if verbose is True:
> > logger.set_verbose()
> >
> > + if re_run < 1:
> > + re_run = 1
> > +
> > logger.log_dir = output_dir
> > log_handler = getLogger('dts')
> > log_handler.config_execution('dts')
> > @@ -520,7 +523,7 @@ def run_all(config_file, pkgName, git, patch,
> > skip_setup,
> >
> > # init dut, tester crb
> > duts, tester = dts_crbs_init(crbInsts, skip_setup,
> > read_cache, project, base_dir, serializer, virttype)
> > -
> > + tester.set_re_run(re_run)
> > # register exit action
> > atexit.register(quit_execution, duts, tester)
> >
> > diff --git a/framework/main.py b/framework/main.py index
> > a018af0..de9d54f 100755
> > --- a/framework/main.py
> > +++ b/framework/main.py
> > @@ -134,6 +134,9 @@ parser.add_argument('--debug',
> > parser.add_argument('--debugcase',
> > action='store_true',
> > help='enable debug mode in the first case, user
> > can further debug')
> > +parser.add_argument('--re_run',
> > + default=1,
> > + help='when case failed will re-run times, and
> > +this
> > value must >= 1')
> >
>
> Suggest add "type=int" in re-run argument definition.
>
> > parser.add_argument('--commands',
> > action='append',
> > @@ -156,4 +159,4 @@ dts.run_all(args.config_file, args.snapshot, args.git,
> > args.patch, args.skip_setup, args.read_cache,
> > args.project, args.suite_dir, args.test_cases,
> > args.dir, args.output, args.verbose,args.virttype,
> > - args.debug, args.debugcase, args.commands)
> > + args.debug, args.debugcase, args.re_run, args.commands)
> > --
> > 1.9.3
More information about the dts
mailing list