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

Issue 595057 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Show all chromiumos.chromium builders in chromiumos announce section.

Project Member Reported by steve...@chromium.org, Mar 15 2016

Issue description

The 'chromiumos.chromium' section of the chromiumos waterfall shows 3 builders:

x86-generic-tot-chromium-pfq-informational
amd64-generic-tot-chromium-pfq-informational
daisy-tot-chromium-pfq-informational

However the waterfall / console view:
https://build.chromium.org/p/chromiumos.chromium/waterfall

Also shows the following builders:

x86-generic-tot-asan-informational
amd64-generic-tot-asan-informational
x86-generic-telemetry
amd64-generic-telemetry

As such, these builders are generally ignored. We should either:
* Add them to the waterfall
* Remove them
* Move them to the i/chromeos.chrome waterfall (even though they are chromiumos builders) if we are concerned about the number of builders on the chromiumos waterfall.

The asan builders have been failing recently (due to a timeout which we will investigate) but have mostly been green.

The telemetry builders have been failing as far back as their history shows. I'm not actually sure what their purpose is.


 

Comment 1 by vapier@chromium.org, Mar 15 2016

Cc: d...@chromium.org

Comment 2 by d...@chromium.org, Mar 15 2016

This has been the state of things for a long time. The builders shown in the announce section are there for sheriffing utility, and there is no intent for parity between the subset that is displayed and the total builders on any given waterfall.

The real question is what role do those other builders play? If CrOS wants to sheriff them, adding them to announce makes sense. If CrOS doesn't care about them, removing them makes sense. I don't think changing their waterfall makes sense, though.

The current situation is that they're there, presumably because they're useful in some capacity, but don't need to be sheriffed.
Discussions on issues tracking current failures in those builders suggest that they should remain active and should be sheriffed (or, more accurately, gardened).

Right now the the -telemetry bots ("perf") are stuck in a failure state and require some debugging to get them green again ( issue 595055 ), but they are in that state because the builders are not discoverable so nobody has been paying attention to them.

The asan-informational builders have a real bug that is being addressed:  issue 595068 .

Comment 4 by d...@chromium.org, Mar 15 2016

Owner: d...@chromium.org
Summary: Show all chromiumos.chromium builders in chromiumos announce section. (was: chromiumos.chromium link only shows certain builders)
Okay, SGTM.
Project Member

Comment 5 by bugdroid1@chromium.org, Mar 15 2016

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

commit 8f72a15627eac19ef94c310d823e8a193f86084c
Author: dnj@chromium.org <dnj@chromium.org>
Date: Tue Mar 15 22:26:08 2016

Show all chromiumos.chromium builders in announce.

TBR=stip@chromium.org
BUG= chromium:595057 
TEST=None

Review URL: https://codereview.chromium.org/1808513002

git-svn-id: svn://svn.chromium.org/chrome/trunk/tools/build@299302 0039d316-1c4b-4281-b951-d872f2087c98

[modify] https://crrev.com/8f72a15627eac19ef94c310d823e8a193f86084c/masters/master.chromiumos/templates/announce.html

Comment 6 by d...@chromium.org, Mar 15 2016

This has been implemented and is now live on the chromiumos waterfall.
Awesome, thanks!

Labels: VerifyIn-51
Status: Verified (was: Fixed)
Bulk verified

Sign in to add a comment