New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 616145 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

chromium.perf.fyi should show up in the list of masters on the waterfall page

Project Member Reported by charliea@chromium.org, May 31 2016

Issue description

aiolos@ and I noticed that chromium.perf.fyi doesn't show up on the list of masters on https://uberchromegw.corp.google.com/. 

sullivan@, dtu@, do either of you know who I should talk to in order to get it added?
 
chromium_perf.png
62.4 KB View Download
Labels: -Infra-Labs Infra
Tagging infra, since infra team does regular triage. Infra team, any idea how to get this listed?

Comment 2 by charliea@google.com, May 31 2016

Doh - sorry, definitely didn't mean to attach Infra labs to this.
Components: Infra
Oh, just realized: should I put this in Infra component or label to get regular triage? Added both for now.
Labels: -Infra
Owner: aga...@chromium.org
Status: Assigned (was: Untriaged)
Assigning to Aaron since he definitely knows how to modify this list...
Yeah, this is just a small modification here: https://chromium.googlesource.com/chromium/tools/build/+/master/masters/master.chromium/templates/announce.html#211

I note that chromium.perf itself isn't listed either... do you want both?
Status: Started (was: Assigned)
https://codereview.chromium.org/2031423002
Project Member

Comment 7 by bugdroid1@chromium.org, Jun 3 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/tools/build.git/+/5f8114f966fcb832c5656881a81635257faba1e7

commit 5f8114f966fcb832c5656881a81635257faba1e7
Author: agable <agable@chromium.org>
Date: Fri Jun 03 18:59:52 2016

Improvements to the announce header.

Removes some dead links, splits masters into masters+tryservers,
and adds some masters which weren't listed previously.

R=vadimsh@chromium.org
BUG= 616145 

Review-Url: https://codereview.chromium.org/2031423002

[modify] https://crrev.com/5f8114f966fcb832c5656881a81635257faba1e7/masters/master.chromium/templates/announce.html

I took a look after the CL and the changes lgtm. Should we mark fixed?
Marking as fixed, unless someone else objects
Status: Fixed (was: Started)
Yes, sorry, I didn't mark fixed because I didn't explicitly deploy this myself. Now that it has been deployed, this is definitely fixed.

Sign in to add a comment