[Telemetry] Move discover to common/py_utils

[Telemetry] Move discover to common/py_utils

BUG=catapult:#3612

Review-Url: https://codereview.chromium.org/2941123002
Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/33a6a6490c5eee24cdccc9692fb0f1b29703fa02

[eakuefner, 2017-06-15 20:31:21]

eakuefner@chromium.org changed reviewers:
+ nednguyen@google.com

[eakuefner, 2017-06-15 20:31:22]

Ned, please take a look.

[nednguyen, 2017-06-15 20:32:35]

On 2017/06/15 20:31:22, eakuefner wrote:
> Ned, please take a look.

discover is used in chromium/src/ so you may need to do a shim, then update the
chromium side, then remove the shim.

[eakuefner, 2017-06-15 20:41:16]

On 2017/06/15 at 20:32:35, nednguyen wrote:
> On 2017/06/15 20:31:22, eakuefner wrote:
> > Ned, please take a look.
> 
> discover is used in chromium/src/ so you may need to do a shim, then update
the chromium side, then remove the shim.

Thanks. As far as I can tell from cs.chromium.org, DiscoverClasses and
DiscoverClassesInModule are the two functions used in chromium/src, so these are
the two that I've shimmed in the latest patchset. I'll also watch the Catapult
roll after this lands in Catapult.

[nednguyen, 2017-06-15 20:44:39]

lgtm

[eakuefner, 2017-06-15 20:46:18]

The CQ bit was checked by eakuefner@chromium.org

[commit-bot: I haz the power, 2017-06-15 20:46:23]

CQ is trying da patch.

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

[eakuefner, 2017-06-15 20:46:33]

The CQ bit was unchecked by eakuefner@chromium.org

[eakuefner, 2017-06-15 20:46:40]

The CQ bit was checked by eakuefner@chromium.org

[commit-bot: I haz the power, 2017-06-15 20:46:44]

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-06-15 21:20:48]

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

[commit-bot: I haz the power, 2017-06-15 21:20:50]

Try jobs failed on following builders:
  Catapult Linux Tryserver on master.tryserver.client.catapult (JOB_FAILED,
https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Li...)

[eakuefner, 2017-06-22 17:22:01]

The CQ bit was checked by eakuefner@chromium.org

[eakuefner, 2017-06-22 17:22:02]

The patchset sent to the CQ was uploaded after l-g-t-m from nednguyen@google.com
Link to the patchset: https://codereview.chromium.org/2941123002/#ps40001
(title: "fix imports")

[commit-bot: I haz the power, 2017-06-22 17:22:10]

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-06-22 17:47:24]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1498152121846310,
"parent_rev": "a857866b281fbbf9ecab0b0fd7f4393f4be91587", "commit_rev":
"33a6a6490c5eee24cdccc9692fb0f1b29703fa02"}

[commit-bot: I haz the power, 2017-06-22 17:47:27]

Description was changed from

==========
[Telemetry] Move discover to common/py_utils

BUG=catapult:#3612
==========

to

==========
[Telemetry] Move discover to common/py_utils

BUG=catapult:#3612

Review-Url: https://codereview.chromium.org/2941123002
Committed:
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...
==========

[commit-bot: I haz the power, 2017-06-22 17:47:28]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...