Update masters templates/announce.html files regarding tree closers |
|||||||||
Issue descriptionIn the view at build.chromium.org there are many bots grouped as "tree closers" that are no longer tree closers. The view should be updated to reflect reality. It's probably best to remove the wording entirely from this page, as it depends on what step fails if the tree is closed (mostly just compile). The file to edit is https://chromium.googlesource.com/chromium/tools/build/+/master/masters/master.chromium/templates/announce.html
,
Sep 1 2016
,
Sep 4 2017
This issue has been Available for over a year. If it's no longer important or seems unlikely to be fixed, please consider closing it out. If it is important, please re-triage the issue. Sorry for the inconvenience if the bug really should have been left as Available. If you change it back, also remove the "Hotlist-Recharge-Cold" label. For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
,
Sep 8 2017
Not sure if is still valid though.
,
Jan 23 2018
Erik, has this changed since we changed to LUCI UI?
,
Jan 23 2018
We copied everything over to avoid changing too many things at once. The config defines the header here: https://chromium.googlesource.com/chromium/src/+/infra/config/luci-milo.cfg I have no objections to changing things around to make more sense. I think this should be up to CCI to decide how they want everything to be displayed so please feel free to do what's best. Assigning back to martiniss, please reassign or make available as needed.
,
Jan 23 2018
We'd need some feature work in LUCI UI to truly fix this bug. The problem here is that the LUCI UI specifies that all builders on chromium.gpu should show up: https://chromium.googlesource.com/chromium/src/+/infra/config/luci-milo.cfg#229 The chromium gatekeeper tree does this as well, but in the chromium.gpu master config in gatekeeper (https://cs.chromium.org/chromium/build/scripts/slave/gatekeeper.json?l=228) it says to exclude several builders. This is the information that milo needs to know about. There are two solutions I can think of: 1. Teach LUCI UI to read the gatekeeper files, and understand the excluded builders thing. 2. Duplicate the list of excluded builders into the luci-milo.cfg file. Need to teach LUCI UI how to exclude builders from a console. Both require some coding as far as I can tell. Doesn't seem like a really trivial change.
,
Jan 24 2018
Instead of (2) we could define a new console (builder group) without the excluded builders and reference that from the header.
,
Mar 9 2018
I don't think I'll have time to do this. I'm going to mark this as available.
,
Mar 12 2018
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by bpastene@chromium.org
, Jun 22 2016