524169 - Enable Talos on the Electrolysis branch

Status: RESOLVED FIXED

(Release Engineering :: General, defect)

Description

[Benjamin Smedberg]

When we started the Electrolysis branch builds there was no point in running Talos on them. Now we're more stable I'd like to start running Talos on these builds on all three desktop platforms. It is not important to run Talos on any mobile platforms at this point (if Talos even exists for any mobile platforms, I'm not clear on that).

Comment 1

[Benjamin Smedberg]

jford, what does the "future" marking mean here? I was told when e10s was created that adding Talos in the future would be really quick and easy.

Comment 2

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

(In reply to comment #1)
> jford, what does the "future" marking mean here? I was told when e10s was
> created that adding Talos in the future would be really quick and easy.

jhford was triaging this, as no-one else had time to deal with this today. I'll take it for now, and thumb-wrestle with lsblakk and anodelman on Monday to see who can deal with this. 

Original comment still stands, yes, this should be easy to enable on this branch. 

Note: we are talking here about enabling the pre-existing talos suites. This is unrelated to the new multi-process talos suites you were asking about elsewhere and which have yet to be written afaik.

Comment 3

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add electrolysis branch to talos config.py

Comment 4

[Benjamin Smedberg]

Yes, existing Talos to match mozilla-central, the other work will come later.

Comment 5

[John O'Duinn [:joduinn] (please use "needinfo?" flag)]

(In reply to comment #4)
> Yes, existing Talos to match mozilla-central, the other work will come later.

Perfect. This we can do quickly, in fact Lukas already has patch up for review.

Comment 6

[Chris AtLee [:catlee]]

Tested in staging?

Comment 7

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add electrolysis branch to talos config.py in staging

Will test this in staging on next downtime.

Comment 8

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Add electrolysis branch to talos config.py in staging

This is the current patch that I ran in staging today. The linux builds have the libstdc++ mismatch at the moment:

mozqa@qm-ref-ubuntu:~/talos-slave/talos-data$ ./firefox/firefox
./firefox/firefox-bin: /usr/lib/libstdc++.so.6: version `GLIBCXX_3.4.10' not found (required by ./firefox/libxul.so)

Comment 9

[Benjamin Smedberg]

Yes. I don't think that should necessarily block putting this into production.

Comment 10

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Enables talos for electrolysis on staging and production

Comment 11

[Lukas Blakk [:lsblakk] use ?needinfo]

We're going to schedule a downtime for this coming Monday, providing the 3.6 release gets the go ahead, and this will get turned on at that time.

Comment 12

[Lukas Blakk [:lsblakk] use ?needinfo]

Downtime update - this Wednesday at 8am PST.

Comment 13

[Lukas Blakk [:lsblakk] use ?needinfo]

Attachment: Turn on talos for Electrolysis with V8 and SVG Opacity

so this one is now queued up behind turning on V8 (bug 506772) and SVG opacity (bug 524089)

Comment 14

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 411550 [details] [diff] [review]
Turn on talos for Electrolysis with V8 and SVG Opacity

http://hg.mozilla.org/build/buildbot-configs/rev/645d3e06adf3

Comment 15

[Lukas Blakk [:lsblakk] use ?needinfo]

Switched this on today, closing.