Pass '--verbose' and '--retries 0' flags to run_test_cases.py for ThreadSanitizer.

Pass '--verbose' and '--retries 0' flags to run_test_cases.py for ThreadSanitizer.
This allows to report a race even in the case it's flaky.

BUG=128314
R=maruel@chromium.org

[Alexander Potapenko, 2013-09-13 15:09:45]

PTAL

[M-A Ruel, 2013-09-13 15:16:25]

https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py
File scripts/slave/runtest.py (right):

https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py#newc...
scripts/slave/runtest.py:1238: if options.sharding_args is None:
Worth using default='' at the options instead.

[M-A Ruel, 2013-09-13 15:17:45]

I meant default=[] but Rietveld is stuck.


2013/9/13 <maruel@chromium.org>

>
> https://codereview.chromium.**org/23532067/diff/1/scripts/**
>
slave/runtest.py<https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py>
> File scripts/slave/runtest.py (right):
>
> https://codereview.chromium.**org/23532067/diff/1/scripts/**
>
slave/runtest.py#newcode1238<https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py#newcode1238>
> scripts/slave/runtest.py:1238: if options.sharding_args is None:
> Worth using default='' at the options instead.
>
>
https://codereview.chromium.**org/23532067/<https://codereview.chromium.org/2...
>

To unsubscribe from this group and stop receiving emails from it, send an email
to chromium-reviews+unsubscribe@chromium.org.

[Alexander Potapenko, 2013-09-13 15:23:08]

Done.

https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py
File scripts/slave/runtest.py (right):

https://codereview.chromium.org/23532067/diff/1/scripts/slave/runtest.py#newc...
scripts/slave/runtest.py:1238: if options.sharding_args is None:
On 2013/09/13 15:16:25, M-A Ruel wrote:
> Worth using default='' at the options instead.

Yeah, I was thinking about that option, but didn't want to change the behavior
in other places. Ok, I'll do that.
This is actually a string, so '' is fine.

[M-A Ruel, 2013-09-13 15:24:05]

https://codereview.chromium.org/23532067/diff/7001/scripts/slave/runtest.py
File scripts/slave/runtest.py (right):

https://codereview.chromium.org/23532067/diff/7001/scripts/slave/runtest.py#n...
scripts/slave/runtest.py:1240: options.sharding_args += '--retries 0'
' --retries 0'

[Alexander Potapenko, 2013-09-13 15:26:20]

https://codereview.chromium.org/23532067/diff/7001/scripts/slave/runtest.py
File scripts/slave/runtest.py (right):

https://codereview.chromium.org/23532067/diff/7001/scripts/slave/runtest.py#n...
scripts/slave/runtest.py:1240: options.sharding_args += '--retries 0'
On 2013/09/13 15:24:05, M-A Ruel wrote:
> ' --retries 0'

Done.

[M-A Ruel, 2013-09-13 15:30:29]

lgtm

[commit-bot: I haz the power, 2013-09-13 15:31:19]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/glider@chromium.org/23532067/13001

[commit-bot: I haz the power, 2013-09-13 15:32:08]

Change committed as 223047