MozillaTry waterfall unusably wide due to talos column bloat

RESOLVED FIXED

Status

Release Engineering
General
RESOLVED FIXED
9 years ago
5 years ago

People

(Reporter: alice, Assigned: alice)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

9 years ago
Each talos box gets it's own column on MozillaTry for each type of test suite it runs - leading to 90 or so columns.
(Assignee)

Updated

9 years ago
Assignee: nobody → anodelman
(Assignee)

Comment 1

9 years ago
Created attachment 395702 [details] [diff] [review]
[checked in]stop lying about talos test start time

I think that this has outlived its usefulness.  We display the changeset of the build being tested in the talos tinderbox waterfall info, so we don't have to line up with the start time of the build being tested.  This will finally do away with the confusion of 'why does talos take so damn long to run' as well.
Attachment #395702 - Flags: review?(bhearsum)
(Assignee)

Comment 2

9 years ago
Created attachment 395703 [details] [diff] [review]
stop using the talos machine name as part of the column name for try

This would be the real meat of the problem on try, in that every machine has its own column for every test that it runs.  Madness!

I've just copied and pasted the mail code from talos-pool and assuming that it will work.  That might be presumptuous.
Attachment #395703 - Flags: review?(bhearsum)
Comment on attachment 395702 [details] [diff] [review]
[checked in]stop lying about talos test start time

happy times!
Attachment #395702 - Flags: review?(bhearsum) → review+
(Assignee)

Comment 4

9 years ago
Created attachment 395982 [details] [diff] [review]
stop using the talos machine name as part of the column name for try

Previous patch was using the wrong mailnotifier.  This set up looks correct to me now.
Attachment #395982 - Flags: review?(bhearsum)
Attachment #395703 - Attachment is obsolete: true
Attachment #395703 - Flags: review?(bhearsum)
Comment on attachment 395982 [details] [diff] [review]
stop using the talos machine name as part of the column name for try

r=bhearsum w/ the unnecessary MailNotifier import removed. I'll land it with that change.
Attachment #395982 - Flags: review?(bhearsum) → review+
Comment on attachment 395982 [details] [diff] [review]
stop using the talos machine name as part of the column name for try

changeset:   1455:6941c851329b
Attachment #395982 - Flags: checked-in+
All the new columns are reporting in now, I just disabled all the slave specific ones.
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
(Assignee)

Comment 8

9 years ago
Failed to land the second patch to stop lying about talos test start time.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 9

9 years ago
Comment on attachment 395702 [details] [diff] [review]
[checked in]stop lying about talos test start time

changeset:   403:aa5d8825346a
Attachment #395702 - Attachment description: stop lying about talos test start time → [checked in]stop lying about talos test start time
Attachment #395702 - Flags: checked-in+
(Assignee)

Comment 10

9 years ago
Additional fix for try talos factory (missed in initial patch):

changeset:   404:d61aac1e02d9
(Assignee)

Comment 11

9 years ago
Talos now reporting correct start time.  No more lies!
Status: REOPENED → RESOLVED
Last Resolved: 9 years ago9 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.