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

Issue 1797303003: Remove telemetry reference builds from chromium DEPS. (Closed)

Created:
4 years, 9 months ago by aiolos (Not reviewing)
Modified:
4 years, 9 months ago
Reviewers:
nednguyen, prasadv, scottmg
CC:
chromium-reviews, auto-bisect-reviews_chromium.org, telemetry-reviews_chromium.org
Base URL:
https://chromium.googlesource.com/chromium/src.git@master
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Remove telemetry reference builds from chromium DEPS. Telemetry is using the DependencyManager to get the reference builds, and no one else is currently using it. Removing it reduces a synced chromium checkout by ~2G on mac. BUG= Committed: https://crrev.com/84be010f617988a6ed1184431930e986ed55f665 Cr-Commit-Position: refs/heads/master@{#381513}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -473 lines) Patch
M DEPS View 3 chunks +0 lines, -9 lines 0 comments Download
M tools/auto_bisect/bisect_utils.py View 1 chunk +0 lines, -3 lines 0 comments Download
M tools/copyright_scanner/copyright_scanner.py View 1 chunk +0 lines, -2 lines 0 comments Download
M tools/export_tarball/export_tarball.py View 1 chunk +0 lines, -1 line 0 comments Download
D tools/perf/update_reference_build.py View 1 chunk +0 lines, -333 lines 0 comments Download
D tools/perf/update_reference_build_unittest.py View 1 chunk +0 lines, -125 lines 0 comments Download

Messages

Total messages: 17 (7 generated)
aiolos (Not reviewing)
prasad: could you review the autobisect changes? (Already talking offline about autobisect presubmit test failures ...
4 years, 9 months ago (2016-03-15 21:33:24 UTC) #3
nednguyen
On 2016/03/15 21:33:24, aiolos wrote: > prasad: could you review the autobisect changes? (Already talking ...
4 years, 9 months ago (2016-03-15 22:10:25 UTC) #4
scottmg
lgtm
4 years, 9 months ago (2016-03-15 22:39:36 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1797303003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1797303003/1
4 years, 9 months ago (2016-03-15 22:54:04 UTC) #7
aiolos (Not reviewing)
Looks like the bisect tests may be broken on mac. Sending through a dry run ...
4 years, 9 months ago (2016-03-15 22:54:19 UTC) #8
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 9 months ago (2016-03-16 00:17:19 UTC) #10
prasadv
lgtm
4 years, 9 months ago (2016-03-16 18:58:11 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1797303003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1797303003/1
4 years, 9 months ago (2016-03-16 19:32:31 UTC) #13
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 9 months ago (2016-03-16 19:42:54 UTC) #15
commit-bot: I haz the power
4 years, 9 months ago (2016-03-16 19:44:42 UTC) #17
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/84be010f617988a6ed1184431930e986ed55f665
Cr-Commit-Position: refs/heads/master@{#381513}

Powered by Google App Engine
This is Rietveld 408576698