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

Issue 2427763002: Remove use of "use_mirror" recipe property. (Closed)

Created:
4 years, 2 months ago by dnj
Modified:
4 years, 2 months ago
Reviewers:
iannucci, martiniss
CC:
chromium-reviews, infra-reviews+build_chromium.org, kjellander-cc_chromium.org
Target Ref:
refs/heads/master
Project:
build
Visibility:
Public.

Description

Remove use of "use_mirror" recipe property. The "use_mirror" property is set by the recipe engine when in a production environment to signal to the "gclient" recipe module that the SVN GOLO mirror should be used instead of the default SVN URL. Since we're not on SVN, this is now meaningless. It will be deprecated in recipe engine, so remove references. The "perf_dashboard" recipe has been using this property as a test to see if it is running on a production system. This test has been replaced with the identical logic that the recipe engine was using to set "use_mirror". A build property has been added to indicate production runs, and a note has been added to indicate that the perf dashboard bot should explicitly specify this as a property rather than infer it from the environment. BUG=chromium:656741 TEST=expectations R=iannucci@chromium.org, martiniss@chromium.org Committed: https://chromium.googlesource.com/chromium/tools/build/+/124f02dd3006293789d17557aea29dfbb8f911e0

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+19 lines, -131 lines) Patch
M scripts/slave/recipe_modules/perf_dashboard/api.py View 2 chunks +9 lines, -1 line 0 comments Download
M scripts/slave/recipe_modules/perf_dashboard/example.py View 1 chunk +4 lines, -3 lines 0 comments Download
M scripts/slave/recipe_modules/perf_dashboard/example.expected/linux.json View 2 chunks +3 lines, -3 lines 0 comments Download
A + scripts/slave/recipe_modules/perf_dashboard/example.expected/linux_testing.json View 0 chunks +-1 lines, --1 lines 0 comments Download
D scripts/slave/recipe_modules/perf_dashboard/example.expected/linux_use_mirror.json View 1 chunk +0 lines, -40 lines 0 comments Download
M scripts/slave/recipe_modules/perf_dashboard/example.expected/mac.json View 2 chunks +3 lines, -3 lines 0 comments Download
A + scripts/slave/recipe_modules/perf_dashboard/example.expected/mac_testing.json View 0 chunks +-1 lines, --1 lines 0 comments Download
D scripts/slave/recipe_modules/perf_dashboard/example.expected/mac_use_mirror.json View 1 chunk +0 lines, -40 lines 0 comments Download
M scripts/slave/recipe_modules/perf_dashboard/example.expected/win.json View 2 chunks +3 lines, -3 lines 0 comments Download
A + scripts/slave/recipe_modules/perf_dashboard/example.expected/win_testing.json View 0 chunks +-1 lines, --1 lines 0 comments Download
D scripts/slave/recipe_modules/perf_dashboard/example.expected/win_use_mirror.json View 1 chunk +0 lines, -40 lines 0 comments Download
M scripts/tools/run_recipe.py View 1 chunk +0 lines, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
dnj
4 years, 2 months ago (2016-10-17 18:41:23 UTC) #1
martiniss
lgtm
4 years, 2 months ago (2016-10-17 18:48:34 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/2427763002/1
4 years, 2 months ago (2016-10-17 18:50:41 UTC) #4
commit-bot: I haz the power
4 years, 2 months ago (2016-10-17 19:11:24 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/tools/build/+/124f02dd3006293789d1...

Powered by Google App Engine
This is Rietveld 408576698