[vpython] Incorporate interpreter path/hash.

[vpython] Incorporate interpreter path/hash.

VirtualEnv bakes the host Python interpreter's path into its
configuration. This means that if the underlying interpreter changes or
is replaced, "vpython" will continue using the old specification
unquestioningly, potentially leading to some inconsistencies.

This approach creates a new message, "Runtime", which represents the
Python runtime configuration, including its interpreter hash. Runtime is
folded into "vpython"'s environment hash, meaning that if the
interpreter changes, "vpython" will assign the new interpreter to a
different environment.

BUG=None
TEST=local

Review-Url: https://codereview.chromium.org/2905943002
Committed: https://github.com/luci/luci-go/commit/6e3d45742db9d47fb9ab3b5f61f4dd6c0f532a1d

[dnj, 2017-05-25 21:39:07]

dnj@chromium.org changed reviewers:
+ iannucci@chromium.org

[dnj, 2017-05-25 21:39:08]

PTAL

[dnj, 2017-05-25 21:39:12]

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

[commit-bot: I haz the power, 2017-05-25 21:39:17]

Dry run: CQ is trying da patch.

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

[iannucci, 2017-05-25 21:45:24]

lgtm

[commit-bot: I haz the power, 2017-05-25 21:46:08]

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

[commit-bot: I haz the power, 2017-05-25 21:46:09]

Dry run: This issue passed the CQ dry run.

[dnj, 2017-05-26 19:54:20]

The CQ bit was checked by dnj@chromium.org

[commit-bot: I haz the power, 2017-05-26 19:54:26]

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-26 20:05:12]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1495828460246760, "parent_rev":
"d06ec5035c609eb030e20b470ce6ceeb0d7f428c", "commit_rev":
"6e3d45742db9d47fb9ab3b5f61f4dd6c0f532a1d"}

[commit-bot: I haz the power, 2017-05-26 20:05:16]

Description was changed from

==========
[vpython] Incorporate interpreter path/hash.

VirtualEnv bakes the host Python interpreter's path into its
configuration. This means that if the underlying interpreter changes or
is replaced, "vpython" will continue using the old specification
unquestioningly, potentially leading to some inconsistencies.

This approach creates a new message, "Runtime", which represents the
Python runtime configuration, including its interpreter hash. Runtime is
folded into "vpython"'s environment hash, meaning that if the
interpreter changes, "vpython" will assign the new interpreter to a
different environment.

BUG=None
TEST=local
==========

to

==========
[vpython] Incorporate interpreter path/hash.

VirtualEnv bakes the host Python interpreter's path into its
configuration. This means that if the underlying interpreter changes or
is replaced, "vpython" will continue using the old specification
unquestioningly, potentially leading to some inconsistencies.

This approach creates a new message, "Runtime", which represents the
Python runtime configuration, including its interpreter hash. Runtime is
folded into "vpython"'s environment hash, meaning that if the
interpreter changes, "vpython" will assign the new interpreter to a
different environment.

BUG=None
TEST=local

Review-Url: https://codereview.chromium.org/2905943002
Committed:
https://github.com/luci/luci-go/commit/6e3d45742db9d47fb9ab3b5f61f4dd6c0f532a1d
==========

[commit-bot: I haz the power, 2017-05-26 20:05:17]

Committed patchset #1 (id:1) as
https://github.com/luci/luci-go/commit/6e3d45742db9d47fb9ab3b5f61f4dd6c0f532a1d