[Predator] Pass config as argument to findit.

[Predator] Pass config as argument to findit.

BUG=659354
Review-Url: https://codereview.chromium.org/2673993003
Committed: https://chromium.googlesource.com/infra/infra/+/d4cc688861fbd3e011dcbeb0f71dda8481577706

[Sharu Jiang, 2017-02-03 23:08:57]

Description was changed from

==========
[Predator] Pass config as argument to findit.

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

to

==========
[Predator] Pass config as argument to findit.

BUG=659354
==========

[Sharu Jiang, 2017-02-03 23:08:57]

katesonia@chromium.org changed reviewers:
+ chanli@chromium.org, mbarbella@chromium.org, wrengr@chromium.org

[chanli, 2017-02-03 23:58:51]

lgtm with a nit

https://codereview.chromium.org/2673993003/diff/1/appengine/findit/crash/find...
File appengine/findit/crash/findit.py (right):

https://codereview.chromium.org/2673993003/diff/1/appengine/findit/crash/find...
appengine/findit/crash/findit.py:45: for ``GitilesRepository``).
nit: add docstring for config

[lijeffrey, 2017-02-06 21:35:33]

lijeffrey@chromium.org changed reviewers:
+ lijeffrey@chromium.org

[lijeffrey, 2017-02-06 21:35:34]

lgtm

[wrengr (wrong one), 2017-02-10 21:58:07]

wrengr@google.com changed reviewers:
+ wrengr@google.com

[wrengr (wrong one), 2017-02-10 21:58:08]

lgtm

[Sharu Jiang, 2017-02-10 23:07:14]

Patchset #3 (id:40001) has been deleted

[Sharu Jiang, 2017-02-10 23:07:38]

https://codereview.chromium.org/2673993003/diff/1/appengine/findit/crash/find...
File appengine/findit/crash/findit.py (right):

https://codereview.chromium.org/2673993003/diff/1/appengine/findit/crash/find...
appengine/findit/crash/findit.py:45: for ``GitilesRepository``).
On 2017/02/03 23:58:50, chanli wrote:
> nit: add docstring for config

Done.

[Sharu Jiang, 2017-02-10 23:08:00]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2017-02-10 23:08:00]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org,
lijeffrey@chromium.org, wrengr@google.com
Link to the patchset: https://codereview.chromium.org/2673993003/#ps60001
(title: "Fix nits.")

[commit-bot: I haz the power, 2017-02-10 23:08:03]

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, 2017-02-10 23:14:32]

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

[commit-bot: I haz the power, 2017-02-10 23:14:33]

Try jobs failed on following builders:
  Infra Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/34431d54f2076d10)

[Sharu Jiang, 2017-02-10 23:49:33]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2017-02-10 23:49:33]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org,
lijeffrey@chromium.org, wrengr@google.com
Link to the patchset: https://codereview.chromium.org/2673993003/#ps80001
(title: "Rebase.")

[commit-bot: I haz the power, 2017-02-10 23:49:36]

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, 2017-02-10 23:56:07]

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

[commit-bot: I haz the power, 2017-02-10 23:56:08]

Try jobs failed on following builders:
  Infra Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3443404b386d4910)

[Sharu Jiang, 2017-02-11 00:06:50]

The CQ bit was checked by katesonia@chromium.org

[Sharu Jiang, 2017-02-11 00:06:51]

The patchset sent to the CQ was uploaded after l-g-t-m from chanli@chromium.org,
lijeffrey@chromium.org, wrengr@google.com
Link to the patchset: https://codereview.chromium.org/2673993003/#ps100001
(title: "Fix pylint.")

[commit-bot: I haz the power, 2017-02-11 00:06:53]

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, 2017-02-11 00:16:10]

CQ is committing da patch.

Bot data: {"patchset_id": 100001, "attempt_start_ts": 1486771610569850,
"parent_rev": "25f9ceb3520ab3410beb92e29d1f4ad9b77c2b8d", "commit_rev":
"d4cc688861fbd3e011dcbeb0f71dda8481577706"}

[commit-bot: I haz the power, 2017-02-11 00:16:13]

Description was changed from

==========
[Predator] Pass config as argument to findit.

BUG=659354
==========

to

==========
[Predator] Pass config as argument to findit.

BUG=659354

Review-Url: https://codereview.chromium.org/2673993003
Committed:
https://chromium.googlesource.com/infra/infra/+/d4cc688861fbd3e011dcbeb0f71dd...
==========

[commit-bot: I haz the power, 2017-02-11 00:16:14]

Committed patchset #5 (id:100001) as
https://chromium.googlesource.com/infra/infra/+/d4cc688861fbd3e011dcbeb0f71dd...