Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=633792
Committed: https://chromium.googlesource.com/infra/infra/+/9aa00e5cc809ffb596567e36a99349c92de2a7a3

[josiahk, 2016-08-03 00:01:47]

Description was changed from

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=
==========

to

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=
==========

[josiahk, 2016-08-03 00:01:47]

josiahk@google.com changed reviewers:
+ chanli@chromium.org, lijeffrey@chromium.org, stgao@chromium.org

[josiahk, 2016-08-03 00:08:31]

Description was changed from

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=
==========

to

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=633792
==========

[josiahk, 2016-08-03 00:09:27]

Hi Jeff, Chan and Shuotao,

Here is the fix for the KeyError issue in try_job_util._NeedANewTryJob!

Thanks!
Josiah

[lijeffrey, 2016-08-03 00:24:09]

The CQ bit was checked by lijeffrey@chromium.org

[lijeffrey, 2016-08-03 00:24:09]

lgtm

[commit-bot: I haz the power, 2016-08-03 00:24:12]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 00:33:36]

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

[commit-bot: I haz the power, 2016-08-03 00:33:37]

Try jobs failed on following builders:
  Infra Mac Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/30669b1692f44210)
  Infra Win Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/30669b167fc35910)

[josiahk, 2016-08-03 00:47:51]

The CQ bit was checked by josiahk@google.com

[commit-bot: I haz the power, 2016-08-03 00:47:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 00:57:28]

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

[commit-bot: I haz the power, 2016-08-03 00:57:29]

Try jobs failed on following builders:
  Infra Mac Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3066b0cf6e20d610)

[josiahk, 2016-08-03 00:59:20]

The CQ bit was checked by josiahk@google.com

[josiahk, 2016-08-03 00:59:21]

The patchset sent to the CQ was uploaded after l-g-t-m from
lijeffrey@chromium.org
Link to the patchset: https://codereview.chromium.org/2203033004/#ps20001
(title: "Merge branch 'master' of https://chromium.googlesource.com/infra/infra
into key-error-fix")

[josiahk, 2016-08-03 00:59:25]

The CQ bit was unchecked by josiahk@google.com

[josiahk, 2016-08-03 00:59:54]

The CQ bit was checked by josiahk@google.com

[commit-bot: I haz the power, 2016-08-03 00:59:58]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[josiahk, 2016-08-03 01:00:00]

The CQ bit was unchecked by josiahk@google.com

[josiahk, 2016-08-03 01:00:31]

The CQ bit was checked by josiahk@google.com

[commit-bot: I haz the power, 2016-08-03 01:00:39]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[josiahk, 2016-08-03 01:00:43]

The CQ bit was unchecked by josiahk@google.com

[josiahk, 2016-08-03 01:00:55]

The CQ bit was checked by josiahk@google.com

[commit-bot: I haz the power, 2016-08-03 01:00:59]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 01:08:52]

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

[commit-bot: I haz the power, 2016-08-03 01:08:53]

Try jobs failed on following builders:
  Infra Mac Tester on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3066bc3f44262110)

[iannucci, 2016-08-03 02:56:09]

The CQ bit was checked by iannucci@chromium.org

[commit-bot: I haz the power, 2016-08-03 02:56:19]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-03 03:09:50]

Description was changed from

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=633792
==========

to

==========
Only run _IsBuildFailureUniqueAcrossPlatforms if need_new_try_job is true.

BUG=633792

Committed:
https://chromium.googlesource.com/infra/infra/+/9aa00e5cc809ffb596567e36a9934...
==========

[commit-bot: I haz the power, 2016-08-03 03:09:52]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/9aa00e5cc809ffb596567e36a9934...