Closed Bug 510992 Opened 13 years ago Closed 13 years ago

MozillaTry waterfall unusably wide due to talos column bloat

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: anodelman, Assigned: anodelman)

Details

Attachments

(2 files, 1 obsolete file)

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: nobody → anodelman
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)
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+
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
Closed: 13 years ago
Resolution: --- → FIXED
Failed to land the second patch to stop lying about talos test start time.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
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+
Additional fix for try talos factory (missed in initial patch):

changeset:   404:d61aac1e02d9
Talos now reporting correct start time.  No more lies!
Status: REOPENED → RESOLVED
Closed: 13 years ago13 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.