Closed
Bug 524169
Opened 16 years ago
Closed 16 years ago
Enable Talos on the Electrolysis branch
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: benjamin, Assigned: lsblakk)
References
Details
Attachments
(3 files, 2 obsolete files)
1.93 KB,
patch
|
anodelman
:
review+
|
Details | Diff | Splinter Review |
3.17 KB,
patch
|
Details | Diff | Splinter Review | |
4.19 KB,
patch
|
anodelman
:
review+
lsblakk
:
checked-in+
|
Details | Diff | Splinter Review |
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).
Updated•16 years ago
|
Component: Release Engineering → Release Engineering: Future
Reporter | ||
Comment 1•16 years ago
|
||
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•16 years ago
|
||
(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.
Assignee: nobody → joduinn
Component: Release Engineering: Future → Release Engineering
Updated•16 years ago
|
OS: Windows NT → All
Assignee | ||
Comment 3•16 years ago
|
||
Attachment #408143 -
Flags: review?(anodelman)
Reporter | ||
Comment 4•16 years ago
|
||
Yes, existing Talos to match mozilla-central, the other work will come later.
Comment 5•16 years ago
|
||
(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.
Assignee: joduinn → lsblakk
Updated•16 years ago
|
Attachment #408143 -
Flags: review?(anodelman) → review+
Comment 6•16 years ago
|
||
Tested in staging?
Assignee | ||
Comment 7•16 years ago
|
||
Will test this in staging on next downtime.
Assignee | ||
Comment 8•16 years ago
|
||
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)
Attachment #408424 -
Attachment is obsolete: true
Reporter | ||
Comment 9•16 years ago
|
||
Yes. I don't think that should necessarily block putting this into production.
Assignee | ||
Comment 10•16 years ago
|
||
Attachment #410400 -
Flags: review?(anodelman)
Assignee | ||
Comment 11•16 years ago
|
||
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.
Updated•16 years ago
|
Attachment #410400 -
Flags: review?(anodelman) → review+
Assignee | ||
Comment 12•16 years ago
|
||
Downtime update - this Wednesday at 8am PST.
Assignee | ||
Comment 13•16 years ago
|
||
so this one is now queued up behind turning on V8 (bug 506772) and SVG opacity (bug 524089)
Attachment #410400 -
Attachment is obsolete: true
Attachment #411550 -
Flags: review?(anodelman)
Updated•16 years ago
|
Attachment #411550 -
Flags: review?(anodelman) → review+
Assignee | ||
Comment 14•16 years ago
|
||
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
Attachment #411550 -
Flags: checked-in+
Assignee | ||
Comment 15•16 years ago
|
||
Switched this on today, closing.
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•