510992 - MozillaTry waterfall unusably wide due to talos column bloat

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[alice nodelman [:alice] [:anode]]

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

Comment 1

[alice nodelman [:alice] [:anode]]

Attachment: [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.

Comment 2

[alice nodelman [:alice] [:anode]]

Attachment: 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.

Comment 3

[[e-mail] (:bhearsum)]

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

happy times!

Comment 4

[alice nodelman [:alice] [:anode]]

Attachment: 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.

Comment 5

[[e-mail] (: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.

Comment 6

[[e-mail] (:bhearsum)]

Comment on attachment 395982 [details] [diff] [review]
stop using the talos machine name as part of the column name for try

changeset:   1455:6941c851329b

Comment 7

[[e-mail] (:bhearsum)]

All the new columns are reporting in now, I just disabled all the slave specific ones.

Comment 8

[alice nodelman [:alice] [:anode]]

Failed to land the second patch to stop lying about talos test start time.

Comment 9

[alice nodelman [:alice] [:anode]]

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

changeset:   403:aa5d8825346a

Comment 10

[alice nodelman [:alice] [:anode]]

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

changeset:   404:d61aac1e02d9

Comment 11

[alice nodelman [:alice] [:anode]]

Talos now reporting correct start time.  No more lies!