[Piglit] [PATCH] framework/run.py: allow additional excluded tests on resume

Mason, Michael W michael.w.mason at intel.com
Wed May 20 14:16:09 PDT 2015


OK, I'm abandoning this patch in favor of this one: 

[Piglit] [PATCH] framework/run.py: add option to not retry incomplete tests on resume

Mike

> -----Original Message-----
> From: Dylan Baker [mailto:baker.dylan.c at gmail.com]
> Sent: Tuesday, May 19, 2015 3:05 PM
> To: Mason, Michael W
> Cc: Daniel Vetter; piglit at lists.freedesktop.org
> Subject: Re: [Piglit] [PATCH] framework/run.py: allow additional excluded tests on resume
> 
> On Tue, May 19, 2015 at 04:45:59PM +0000, Mason, Michael W wrote:
> > How about something like this instead of my original patch? I think we
> > should preserve the fact that some tests didn't complete. I'll submit
> > this in a separate email if you all agree.
> >
> > --- a/framework/programs/run.py
> > +++ b/framework/programs/run.py
> > @@ -313,6 +313,9 @@ def resume(input_):
> >                          type=argparse.FileType("r"),
> >                          help="Optionally specify a piglit config file to use. "
> >                               "Default is piglit.conf")
> > +    parser.add_argument("-n", "--no-retry", dest='no_retry',
> 
> Drop dest down to a new line like the others please.
> 
> > +                        action='store_true',
> > +                        help="Do not retry incomplete tests")
> >      args = parser.parse_args(input_)
> >      _disable_windows_exception_messages()
> >
> > @@ -342,7 +345,7 @@ def resume(input_):
> >      # Don't re-run tests that have already completed, incomplete status tests
> >      # have obviously not completed.
> >      for name, result in results.tests.iteritems():
> > -        if result['result'] != 'incomplete':
> > +        if result['result'] != 'incomplete' or args.no_retry:
> 
> It really doesn't matter but, should we check args.no_retry first? Since python is lazy I think it will be optimal in most cases.
> 
> >              opts.exclude_tests.add(name)
> >
> This looks reasonable to me. I left a couple of nits above.
> 
> Dylan


More information about the Piglit mailing list