Add --retry_failed=3 flag to gtest-parallel.

Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max, one run and three retries. 

BUG=chromium:718339
Review-Url: https://codereview.webrtc.org/2861043002
Cr-Commit-Position: refs/heads/master@{#18028}
Committed: https://chromium.googlesource.com/external/webrtc/+/2a28035627ab93463005d9549b2f47db6600aa69

[ehmaldonado_webrtc, 2017-05-04 20:25:27]

The CQ bit was checked by ehmaldonado@webrtc.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-04 20:25:32]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-05-04 20:41:48]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-05-04 20:41:49]

Dry run: This issue passed the CQ dry run.

[ehmaldonado_webrtc, 2017-05-05 08:23:15]

ehmaldonado@webrtc.org changed reviewers:
+ kjellander@webrtc.org

[kjellander_webrtc, 2017-05-05 08:32:59]

lgtm, but can you make it clear what this mean?
Does it mean it will do three retry attempts in addition to the first failing
one i.e. max 4 runs in total, or is it max 3 runs in total?

[ehmaldonado_webrtc, 2017-05-05 09:37:15]

Description was changed from

==========
Add --retry_failed=3 flag to gtest-parallel.

BUG=chromium:718339
==========

to

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max. One run and three re-tries. 

BUG=chromium:718339
==========

[ehmaldonado_webrtc, 2017-05-05 09:37:57]

On 2017/05/05 08:32:59, kjellander_webrtc wrote:
> lgtm, but can you make it clear what this mean?
> Does it mean it will do three retry attempts in addition to the first failing
> one i.e. max 4 runs in total, or is it max 3 runs in total?

Max. 4 runs in total. One run and three retries.

[ehmaldonado_webrtc, 2017-05-05 09:37:59]

On 2017/05/05 08:32:59, kjellander_webrtc wrote:
> lgtm, but can you make it clear what this mean?
> Does it mean it will do three retry attempts in addition to the first failing
> one i.e. max 4 runs in total, or is it max 3 runs in total?

Max. 4 runs in total. One run and three retries.

[ehmaldonado_webrtc, 2017-05-05 09:38:07]

Description was changed from

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max. One run and three re-tries. 

BUG=chromium:718339
==========

to

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max. One run and three retries. 

BUG=chromium:718339
==========

[ehmaldonado_webrtc, 2017-05-05 09:38:30]

Description was changed from

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max. One run and three retries. 

BUG=chromium:718339
==========

to

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max, one run and three retries. 

BUG=chromium:718339
==========

[ehmaldonado_webrtc, 2017-05-05 11:23:42]

On 2017/05/05 09:37:59, ehmaldonado_webrtc wrote:
> On 2017/05/05 08:32:59, kjellander_webrtc wrote:
> > lgtm, but can you make it clear what this mean?
> > Does it mean it will do three retry attempts in addition to the first
failing
> > one i.e. max 4 runs in total, or is it max 3 runs in total?
> 
> Max. 4 runs in total. One run and three retries.

So, is it OK as is?

[kjellander_webrtc, 2017-05-05 11:28:10]

On 2017/05/05 11:23:42, ehmaldonado_webrtc wrote:
> On 2017/05/05 09:37:59, ehmaldonado_webrtc wrote:
> > On 2017/05/05 08:32:59, kjellander_webrtc wrote:
> > > lgtm, but can you make it clear what this mean?
> > > Does it mean it will do three retry attempts in addition to the first
> failing
> > > one i.e. max 4 runs in total, or is it max 3 runs in total?
> > 
> > Max. 4 runs in total. One run and three retries.
> 
> So, is it OK as is?

yes

[kjellander_webrtc, 2017-05-05 11:28:10]

On 2017/05/05 11:23:42, ehmaldonado_webrtc wrote:
> On 2017/05/05 09:37:59, ehmaldonado_webrtc wrote:
> > On 2017/05/05 08:32:59, kjellander_webrtc wrote:
> > > lgtm, but can you make it clear what this mean?
> > > Does it mean it will do three retry attempts in addition to the first
> failing
> > > one i.e. max 4 runs in total, or is it max 3 runs in total?
> > 
> > Max. 4 runs in total. One run and three retries.
> 
> So, is it OK as is?

yes

[ehmaldonado_webrtc, 2017-05-05 11:31:16]

The CQ bit was checked by ehmaldonado@webrtc.org

[commit-bot: I haz the power, 2017-05-05 11:31:26]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.webrtc.org/...

[commit-bot: I haz the power, 2017-05-05 11:33:58]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1493983876882860, "parent_rev":
"3568d8d4d47e9a38d6914261a7c14ec4ee805ed3", "commit_rev":
"2a28035627ab93463005d9549b2f47db6600aa69"}

[commit-bot: I haz the power, 2017-05-05 11:34:01]

Description was changed from

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max, one run and three retries. 

BUG=chromium:718339
==========

to

==========
Add --retry_failed=3 flag to gtest-parallel.

This means the test will be executed 4 times max, one run and three retries. 

BUG=chromium:718339

Review-Url: https://codereview.webrtc.org/2861043002
Cr-Commit-Position: refs/heads/master@{#18028}
Committed:
https://chromium.googlesource.com/external/webrtc/+/2a28035627ab93463005d9549...
==========

[commit-bot: I haz the power, 2017-05-05 11:34:02]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/external/webrtc/+/2a28035627ab93463005d9549...