[CRIU] [PATCH] zdtm.py: add option --ignore-fails

Pavel Emelyanov xemul at virtuozzo.com
Mon Mar 14 15:18:08 PDT 2016


>> The 'or os.WEXITSTATUS(status) check becomes unneeded here.
>>
>>> -					status = 1
>>> +					status = os.WEXITSTATUS(status)
>>>  				break;
>>>  
>>>  	sys.exit(status)
>>> @@ -1427,6 +1433,7 @@ rp.add_argument("--dry-run", help="Don't run tests, just pretend to", action='st
>>>  rp.add_argument("-k", "--keep-img", help = "Whether or not to keep images after test",
>>>  		choices = [ 'always', 'never', 'failed' ], default = 'failed')
>>>  rp.add_argument("--report", help = "Generate summary report in directory")
>>> +rp.add_argument("--ignore-fails", help = "Ignore fails and run all planned tests", action = 'store_true')
>>
>> I don't insist, but Ivan's version of the option name (--keep-going) looks nicer for me.
> 
> It's up to you :) I don't mind.
> Should I use description for option from Ivan's version or keep it?

Up to you :)

>>>  lp = sp.add_parser("list", help = "List tests")
>>>  lp.set_defaults(action = list_tests)
> 



More information about the CRIU mailing list