chromium.perf.fyi should show up in the list of masters on the waterfall page |
||||
Issue descriptionaiolos@ 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?
,
May 31 2016
Doh - sorry, definitely didn't mean to attach Infra labs to this.
,
May 31 2016
Oh, just realized: should I put this in Infra component or label to get regular triage? Added both for now.
,
Jun 3 2016
Assigning to Aaron since he definitely knows how to modify this list...
,
Jun 3 2016
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?
,
Jun 3 2016
,
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
,
Jun 3 2016
I took a look after the CL and the changes lgtm. Should we mark fixed?
,
Jun 13 2016
Marking as fixed, unless someone else objects
,
Jun 14 2016
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 |
||||
Comment 1 by sullivan@chromium.org
, May 31 2016