Closed
Bug 1080660
Opened 12 years ago
Closed 12 years ago
turn on talos for win8.x64 on the 'Date' tree
Categories
(Release Engineering :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: armenzg)
References
Details
Attachments
(2 files, 1 obsolete file)
|
790 bytes,
patch
|
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
|
1.18 KB,
patch
|
rail
:
review+
armenzg
:
checked-in+
|
Details | Diff | Splinter Review |
somehow on date, the win8 opt (32 bit code) is running talos, but the win8 64 bit code is running unittests, and not talos.
Here is the config for date:
http://hg.mozilla.org/build/buildbot-configs/file/1cc91f73ddca/mozilla/project_branches.py#l154
it doesn't indicate we are running talos on win32 and it says we are running unittests, but I don't see those.
| Reporter | ||
Comment 1•12 years ago
|
||
from my understanding of the project branch and what I see running on date https://tbpl-dev.allizom.org/?tree=Date, this patch shouldn't work. But it is all I can do with the information I can find.
| Reporter | ||
Comment 2•12 years ago
|
||
thinking about this more, we really should turn off win8 (32 bit) talos so we can easily compare results from the Date branch to inbound.
| Assignee | ||
Comment 3•12 years ago
|
||
(In reply to Joel Maher (:jmaher) from comment #2)
> thinking about this more, we really should turn off win8 (32 bit) talos so
> we can easily compare results from the Date branch to inbound.
You're right.
I will start working on this tomorrow.
Assignee: jmaher → armenzg
Comment 4•12 years ago
|
||
Yes, that will happen in bug 1080134 as well I imagine.
| Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 8502572 [details] [diff] [review]
a blind view to turn on talos for date win8 64 bit (1.0)
I started working on this. This patch does not do the trick.
I will have something ready for review on Tuesday (Monday is off).
Attachment #8502572 -
Attachment is obsolete: true
Attachment #8502572 -
Flags: review?(catlee)
| Assignee | ||
Comment 6•12 years ago
|
||
We have enabled talos on Win64 for date through this change:
http://hg.mozilla.org/build/buildbot-configs/rev/b7d9a33e1875#l2.13
I have used trigger_arbitrary_build.py to trigger one win64 talos job:
https://tbpl.mozilla.org/?tree=Date&jobname=talos
I will have to reboot into my Win8 machine to see try to run talos.
My attachment disables talos for Win8 32-bit (aka WINNT 6.2) [1].
We don't want to run both side by side as they collide in graphs.m.o.
[1]
Builders removed
- WINNT 6.2 date talos chromez
- WINNT 6.2 date talos dromaeojs
- WINNT 6.2 date talos g1
- WINNT 6.2 date talos other_nol64
- WINNT 6.2 date talos svgr
- WINNT 6.2 date talos tp5o
Attachment #8504910 -
Flags: review?(rail)
| Assignee | ||
Comment 7•12 years ago
|
||
We don't have the same path as the win64 vms.
Attachment #8504970 -
Flags: review?(rail)
Updated•12 years ago
|
Attachment #8504910 -
Flags: review?(rail) → review+
Comment 8•12 years ago
|
||
Comment on attachment 8504970 [details] [diff] [review]
change_win64_talos_python_path.diff
OUCH!
cltbld@T-W864-IX-042 /c/mozilla-build
$ c:/python27/python --version
bash: c:/python27/python: No such file or directory
phew!
cltbld@T-W864-IX-042 /c/mozilla-build
$ c:/mozilla-build/python27/python --version
Python 2.7.3
Thanks for fixing this!
Attachment #8504970 -
Flags: review?(rail) → review+
| Assignee | ||
Comment 9•12 years ago
|
||
Comment on attachment 8504910 [details] [diff] [review]
talos_win64_date.diff
http://hg.mozilla.org/build/buildbot-configs/rev/048d6dd35ab3
Attachment #8504910 -
Flags: checked-in+
| Assignee | ||
Comment 10•12 years ago
|
||
Comment on attachment 8504970 [details] [diff] [review]
change_win64_talos_python_path.diff
http://hg.mozilla.org/build/buildbot-configs/rev/cf7b5748c719
Attachment #8504970 -
Flags: checked-in+
| Assignee | ||
Comment 11•12 years ago
|
||
We have them running on Date.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•8 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•