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

Issue 2473243003: swarmbucket: git_cache must be in cwd (Closed)

Created:
4 years, 1 month ago by nodir
Modified:
4 years, 1 month ago
Reviewers:
Vadim Sh., martiniss
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M recipe_modules/infra_paths/path_config.py View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 8 (2 generated)
nodir
PTAL
4 years, 1 month ago (2016-11-04 23:01:42 UTC) #1
Vadim Sh.
rs lgtm
4 years, 1 month ago (2016-11-04 23:17:01 UTC) #2
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/2473243003/1
4 years, 1 month ago (2016-11-04 23:17:38 UTC) #4
martiniss
lgtm
4 years, 1 month ago (2016-11-04 23:18:55 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/depot_tools/+/dfe6bbbb22379ab9237e75597fd7244e56dd630c
4 years, 1 month ago (2016-11-04 23:21:39 UTC) #7
nodir
4 years, 1 month ago (2016-11-05 01:03:23 UTC) #8
Message was sent while issue was closed.
On 2016/11/04 23:21:39, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/tools/depot_tools/+/dfe6bbbb22379a...

It worked as intended, look at the cache dir
https://luci-milo.appspot.com/swarming/task/324ad126f04ac110/steps/bot_update...

Powered by Google App Engine
This is Rietveld 408576698