Add vpython support to Python recipe module.

Add vpython support to Python recipe module.

Add a keyword argument to the Python recipe module's "__call__" method
that can be used to configure the VirtualEnv under which the module is
run.

BUG=None
TEST=expectations

Review-Url: https://codereview.chromium.org/2875823002
Committed: https://github.com/luci/recipes-py/commit/60cc7f3f49737e8cf852e396b4436c2b6bd3f56d

[dnj (Google), 2017-05-10 21:53:13]

dnj@google.com changed reviewers:
+ dnj@google.com, iannucci@chromium.org

[dnj (Google), 2017-05-10 21:53:13]

PTAL

[iannucci, 2017-05-10 22:46:14]

lgtm

https://codereview.chromium.org/2875823002/diff/1/recipe_modules/python/api.py
File recipe_modules/python/api.py (right):

https://codereview.chromium.org/2875823002/diff/1/recipe_modules/python/api.p...
recipe_modules/python/api.py:17: script (str): The name of the script to run (or
the first command-line
Path

also, TODO: wtf

https://codereview.chromium.org/2875823002/diff/1/recipe_modules/python/examp...
File recipe_modules/python/example.py (right):

https://codereview.chromium.org/2875823002/diff/1/recipe_modules/python/examp...
recipe_modules/python/example.py:35: api.python('run vpython.script',
api.resource('test.py'), venv=True)
let's do test_embedded.py and test_external.py

[dnj (Google), 2017-05-10 22:59:29]

The CQ bit was checked by dnj@google.com

[dnj (Google), 2017-05-10 22:59:29]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2875823002/#ps20001
(title: "comments")

[commit-bot: I haz the power, 2017-05-10 22:59:34]

CQ is trying da patch.

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

[dnj (Google), 2017-05-10 23:04:55]

The CQ bit was unchecked by dnj@google.com

[dnj (Google), 2017-05-10 23:12:05]

The CQ bit was checked by dnj@google.com to run a CQ dry run

[commit-bot: I haz the power, 2017-05-10 23:12:13]

Dry run: 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-05-10 23:18:23]

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

[commit-bot: I haz the power, 2017-05-10 23:18:24]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/360d771b81676010)

[commit-bot: I haz the power, 2017-05-10 23:18:43]

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

[commit-bot: I haz the power, 2017-05-10 23:18:44]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/360d771b81676010)

[dnj, 2017-05-11 15:05:48]

The CQ bit was checked by dnj@chromium.org to run a CQ dry run

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

Dry run: 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-05-11 15:08:28]

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

[commit-bot: I haz the power, 2017-05-11 15:08:29]

Dry run: Try jobs failed on following builders:
  Recipes-py Presubmit on luci.infra.try (JOB_FAILED,
https://luci-milo.appspot.com/swarming/task/3610de72420c5510)

[dnj, 2017-05-11 15:20:44]

The CQ bit was checked by dnj@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-05-11 15:20:48]

Dry run: 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-05-11 15:23:57]

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

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

Dry run: This issue passed the CQ dry run.

[iannucci, 2017-05-12 07:53:35]

let's land this tomorrow?

[dnj, 2017-05-12 15:57:44]

The CQ bit was checked by dnj@chromium.org

[dnj, 2017-05-12 15:57:44]

The patchset sent to the CQ was uploaded after l-g-t-m from
iannucci@chromium.org
Link to the patchset: https://codereview.chromium.org/2875823002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2017-05-12 15:57: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, 2017-05-12 16:00:48]

CQ is committing da patch.

Bot data: {"patchset_id": 60001, "attempt_start_ts": 1494604664414970,
"parent_rev": "b7c18e0258998652eeb302c83680f6a2b1aee398", "commit_rev":
"60cc7f3f49737e8cf852e396b4436c2b6bd3f56d"}

[commit-bot: I haz the power, 2017-05-12 16:00:51]

Description was changed from

==========
Add vpython support to Python recipe module.

Add a keyword argument to the Python recipe module's "__call__" method
that can be used to configure the VirtualEnv under which the module is
run.

BUG=None
TEST=expectations
==========

to

==========
Add vpython support to Python recipe module.

Add a keyword argument to the Python recipe module's "__call__" method
that can be used to configure the VirtualEnv under which the module is
run.

BUG=None
TEST=expectations

Review-Url: https://codereview.chromium.org/2875823002
Committed:
https://github.com/luci/recipes-py/commit/60cc7f3f49737e8cf852e396b4436c2b6bd...
==========

[commit-bot: I haz the power, 2017-05-12 16:00:52]

Committed patchset #4 (id:60001) as
https://github.com/luci/recipes-py/commit/60cc7f3f49737e8cf852e396b4436c2b6bd...