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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 2180343002: Reland of [Telemetry] Remove BuildbotOutputFormatter (Closed)

Created:
4 years, 4 months ago by nednguyen
Modified:
4 years, 4 months ago
CC:
catapult-reviews_chromium.org, Camillo Bruni, kouhei (in TOK), perezju, petrcermak, Primiano Tucci (use gerrit), rnephew (Reviews Here), telemetry-reviews_chromium.org, ulan, vmiura
Base URL:
git@github.com:catapult-project/catapult.git@master
Target Ref:
refs/heads/master
Project:
catapult
Visibility:
Public.

Description

Reland of [Telemetry] Remove BuildbotOutputFormatter (patchset #1 id:1 of https://codereview.chromium.org/2064753008/ ) Reason for revert: crbug.com/620464 is fixed. There should be no one who still actively uses BuildbotOutputFormatter. Removal notice: https://groups.google.com/a/chromium.org/forum/#!topic/telemetry-announce/UVrK9pvRC7I Original issue's description: > Revert of [Telemetry] Remove BuildbotOutputFormatter (patchset #1 id:1 of https://codereview.chromium.org/2012793002/ ) > > Reason for revert: > Other team is still using it. > > Original issue's description: > > [Telemetry] Remove BuildbotOutputFormatter > > > > BUG=chromium:477056 > > > > Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/0fdcac0b5843b7002bee2f87b9af2f8d96ddba06 > > TBR=nednguyen@chromium.org,prasadv@chromium.org,robertocn@chromium.org,eakuefner@chromium.org > # Not skipping CQ checks because original CL landed more than 1 days ago. > BUG=chromium:477056 > > Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/d1858050f3e805c900f1628f34d90c13d67b4cf6 TBR=nednguyen@chromium.org,prasadv@chromium.org,robertocn@chromium.org,eakuefner@chromium.org,achuith@chromium.org # Not skipping CQ checks because original CL landed more than 1 days ago. BUG=chromium:477056 Committed: https://chromium.googlesource.com/external/github.com/catapult-project/catapult/+/bd5d7db6e8d39b18ea49ffc50b066b5b682144e6

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+3 lines, -180 lines) Patch
D telemetry/telemetry/internal/results/buildbot_output_formatter.py View 1 chunk +0 lines, -120 lines 0 comments Download
D telemetry/telemetry/internal/results/buildbot_output_formatter_unittest.py View 1 chunk +0 lines, -42 lines 0 comments Download
M telemetry/telemetry/internal/results/results_options.py View 1 5 chunks +3 lines, -18 lines 0 comments Download

Messages

Total messages: 10 (5 generated)
nednguyen
Created Reland of [Telemetry] Remove BuildbotOutputFormatter
4 years, 4 months ago (2016-07-26 18:08:46 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/2180343002/1
4 years, 4 months ago (2016-08-03 10:13:46 UTC) #4
commit-bot: I haz the power
Try jobs failed on following builders: Catapult Android Tryserver on master.tryserver.client.catapult (JOB_FAILED, https://build.chromium.org/p/tryserver.client.catapult/builders/Catapult%20Android%20Tryserver/builds/650) Catapult Linux ...
4 years, 4 months ago (2016-08-03 10:14:56 UTC) #6
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/2180343002/90001
4 years, 4 months ago (2016-08-03 12:27:09 UTC) #8
commit-bot: I haz the power
4 years, 4 months ago (2016-08-03 12:52:13 UTC) #10
Message was sent while issue was closed. 
Committed patchset #2 (id:90001) as
https://chromium.googlesource.com/external/github.com/catapult-project/catapu...

Powered by Google App Engine
This is Rietveld 408576698