Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(263)

Issue 2107283002: kitchen: add -python-path (Closed)

Created:
4 years, 5 months ago by nodir
Modified:
4 years, 5 months ago
Reviewers:
estaab
CC:
andrew.wang, chromium-reviews, infra-reviews+luci-go_chromium.org, M-A Ruel, tandrii+luci-go_chromium.org, todd
Base URL:
https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Target Ref:
refs/heads/master
Project:
luci-go
Visibility:
Public.

Description

kitchen: add -python-path Problem: swarming_task_template specified PYTHONPATH env variable to a path relative to run dir because it cannot specify absolute path. It breaks during task execution as soon as cwd is changed, e.g. when kitchen runs recipes.py. Fixing this problem at swarming level is a bit controversial: special treatment of PYTHONPATH would be more special in swarming than in kitchen because swarming is more general than kitchen. Fix it by adding explicit support for python paths in kitchen: make them absolute and set env variable. R=estaab@chromium.org BUG=623269 Committed: https://github.com/luci/luci-go/commit/b1931833114a989a4e278398fd214dea83d1ca89

Patch Set 1 #

Patch Set 2 : kitchen: add -python-path #

Patch Set 3 : rebased #

Unified diffs Side-by-side diffs Delta from patch set Stats (+30 lines, -34 lines) Patch
M client/cmd/kitchen/cook.go View 1 5 chunks +15 lines, -0 lines 0 comments Download
A + common/flag/stringlistflag/stringlistflag.go View 3 chunks +8 lines, -20 lines 0 comments Download
A + common/flag/stringlistflag/stringlistflag_test.go View 2 chunks +6 lines, -12 lines 0 comments Download
M common/flag/stringsetflag/stringsetflag_test.go View 1 chunk +1 line, -2 lines 0 comments Download

Messages

Total messages: 17 (8 generated)
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2107283002/40001
4 years, 5 months ago (2016-06-29 16:41:49 UTC) #4
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: Luci-go Linux Precise 32 Tester (Swarming) on ...
4 years, 5 months ago (2016-06-29 16:46:40 UTC) #6
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2107283002/40001
4 years, 5 months ago (2016-06-29 17:09:27 UTC) #8
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 5 months ago (2016-06-29 17:19:43 UTC) #10
nodir
Ptal
4 years, 5 months ago (2016-06-30 14:31:21 UTC) #11
nodir
ping
4 years, 5 months ago (2016-07-01 16:11:26 UTC) #12
estaab
lgtm
4 years, 5 months ago (2016-07-01 18:46:05 UTC) #13
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2107283002/40001
4 years, 5 months ago (2016-07-01 20:10:24 UTC) #15
commit-bot: I haz the power
4 years, 5 months ago (2016-07-01 20:19:21 UTC) #17
Message was sent while issue was closed.
Committed patchset #3 (id:40001) as
https://github.com/luci/luci-go/commit/b1931833114a989a4e278398fd214dea83d1ca89

Powered by Google App Engine
This is Rietveld 408576698