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

Issue 2176913002: infra_paths module: swarmbucket config (Closed)

Created:
4 years, 5 months ago by nodir
Modified:
4 years, 5 months ago
Reviewers:
dnj
CC:
chromium-reviews, dpranke+depot_tools_chromium.org, iannucci+depot_tools_chromium.org
Target Ref:
refs/heads/master
Visibility:
Public.

Description

infra_paths module: swarmbucket config add infra_swarmbucket config for infra_paths module to override git_cache. The value of path_config property value will be specified by kitchen. This will still delete cache every time. The goal of this change is to fix build.git swarmbucket builds. TBR=dnj@chromium.org BUG= Committed: https://chromium.googlesource.com/chromium/tools/depot_tools/+/248331450c05c59c8e966c806f00bd2475e36603

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, --2 lines) Patch
M recipe_modules/infra_paths/example.py View 1 chunk +1 line, -1 line 0 comments Download
A + recipe_modules/infra_paths/example.expected/paths_swarmbucket_linux.json View 0 chunks +-1 lines, --1 lines 0 comments Download
A + recipe_modules/infra_paths/example.expected/paths_swarmbucket_mac.json View 0 chunks +-1 lines, --1 lines 0 comments Download
A + recipe_modules/infra_paths/example.expected/paths_swarmbucket_win.json View 0 chunks +-1 lines, --1 lines 0 comments Download
M recipe_modules/infra_paths/path_config.py View 1 chunk +5 lines, -0 lines 0 comments Download

Messages

Total messages: 7 (3 generated)
nodir
PTAL
4 years, 5 months ago (2016-07-22 23:33:03 UTC) #1
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/2176913002/1
4 years, 5 months ago (2016-07-22 23:44:54 UTC) #4
commit-bot: I haz the power
Committed patchset #1 (id:1) as https://chromium.googlesource.com/chromium/tools/depot_tools/+/248331450c05c59c8e966c806f00bd2475e36603
4 years, 5 months ago (2016-07-22 23:47:59 UTC) #6
dnj
4 years, 5 months ago (2016-07-22 23:52:34 UTC) #7
Message was sent while issue was closed.
lgtm

Powered by Google App Engine
This is Rietveld 408576698