Add 'try_job_id' to bisect option.

Add 'try_job_id' to bisect option.

BUG=477448
Committed: https://crrev.com/2a334363d09f2a3c454a25924f226b161b4c2e5e
Cr-Commit-Position: refs/heads/master@{#383550}

[chrisphan, 2016-03-22 23:55:14]

chrisphan@chromium.org changed reviewers:
+ qyearsley@chromium.org

[qyearsley, 2016-03-23 02:28:48]

LGTM - Aye, adding new options to the old bisect config requires several little
changes :-/

If legacy bisect is triggered via bisect.cfg and run-bisect-perf-regression.py,
then the optional also needs to be added to
https://code.google.com/p/chromium/codesearch#chromium/src/tools/run-bisect-p...
-- maybe it would be good to add that in this same CL?

[chrisphan, 2016-03-24 00:25:08]

On 2016/03/23 02:28:48, qyearsley wrote:
> LGTM - Aye, adding new options to the old bisect config requires several
little
> changes :-/
> 
> If legacy bisect is triggered via bisect.cfg and
run-bisect-perf-regression.py,
> then the optional also needs to be added to
>
https://code.google.com/p/chromium/codesearch#chromium/src/tools/run-bisect-p...
> -- maybe it would be good to add that in this same CL?

Ah thanks for that.  Done.

[chrisphan, 2016-03-24 00:25:31]

chrisphan@chromium.org changed reviewers:
+ prasadv@chromium.org

[chrisphan, 2016-03-24 00:25:32]

Prasadv can you double check for me this is correct?  Thanks

[chrisphan, 2016-03-24 18:58:53]

The CQ bit was checked by chrisphan@chromium.org

[chrisphan, 2016-03-24 18:58:53]

The patchset sent to the CQ was uploaded after l-g-t-m from
qyearsley@chromium.org
Link to the patchset:
https://chromiumcodereview-hr.appspot.com/1823803004/#ps20001 (title: "add
try_job_id to run-bisect-perf-regression.py")

[commit-bot: I haz the power, 2016-03-24 18:59:11]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823803004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823803004/20001

[commit-bot: I haz the power, 2016-03-24 19:18:00]

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

[commit-bot: I haz the power, 2016-03-24 19:18:00]

Try jobs failed on following builders:
  chromium_presubmit on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[chrisphan, 2016-03-24 19:19:21]

Oh, going to need your LGTM Prasad.  Thanks

[prasadv, 2016-03-28 19:03:24]

lgtm

[chrisphan, 2016-03-28 19:03:44]

The CQ bit was checked by chrisphan@chromium.org

[commit-bot: I haz the power, 2016-03-28 19:04:07]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1823803004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1823803004/20001

[commit-bot: I haz the power, 2016-03-28 20:20:04]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-28 20:21:00]

Description was changed from

==========
Add 'try_job_id' to bisect option.

BUG=477448
==========

to

==========
Add 'try_job_id' to bisect option.

BUG=477448

Committed: https://crrev.com/2a334363d09f2a3c454a25924f226b161b4c2e5e
Cr-Commit-Position: refs/heads/master@{#383550}
==========

[commit-bot: I haz the power, 2016-03-28 20:21:01]

Patchset 2 (id:??) landed as
https://crrev.com/2a334363d09f2a3c454a25924f226b161b4c2e5e
Cr-Commit-Position: refs/heads/master@{#383550}