Closed
Bug 607910
Opened 14 years ago
Closed 14 years ago
[SeaMonkey 2.0, Windows] Packaged tests are not run anymore
Categories
(SeaMonkey :: Release Engineering, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: sgautherie, Assigned: kairo)
References
(Blocks 1 open bug)
Details
(Keywords: regression)
Attachments
(1 file)
1.86 KB,
patch
|
gozer
:
review+
|
Details | Diff | Splinter Review |
Since at least 2010/09/17 :-(
Flags: in-testsuite-
Reporter | ||
Comment 1•14 years ago
|
||
I checked config.py but didn't notice anything unexpected there.
I have no clue.
![]() |
Assignee | |
Comment 2•14 years ago
|
||
The columns are there on the buildbot waterfall, I guess they are not triggered any more though.
![]() |
Assignee | |
Comment 3•14 years ago
|
||
WINNT 5.2 comm-1.9.1 unit test: steps/sendchange/logs/stdio has
python /e/builds/slave/comm-1.9.1-win32-unittest/tools/buildfarm/utils/retry.py -s 5 -t 1800 -r 5 --stdout-regexp change sent successfully buildbot sendchange --master cb-seamonkey-linuxmaster-01.mozilla.org:9010 --username sendchange-unittest --branch comm-1.9.1-win32-unittest --revision None http://stage.mozilla.org/pub/mozilla.org/seamonkey/tinderbox-builds/comm-1.9.1-win32-unittest/1288252726/seamonkey-2.0.10pre.en-US.win32.zip
in dir e:\builds\slave\comm-1.9.1-win32-unittest\build (timeout 1200 secs)
[...]python: can't open file '/e/builds/slave/comm-1.9.1-win32-unittest/tools/buildfarm/utils/retry.py': [Errno 2] No such file or directory
program finished with exit code 2
![]() |
Assignee | |
Comment 4•14 years ago
|
||
I tried to solve this with http://hg.mozilla.org/build/buildbotcustom/rev/559708821cb9
![]() |
Assignee | |
Comment 5•14 years ago
|
||
For correctness, seeking review for this patch. The reason it should help is because the path we call retry.py with only differs to the debug builds that invoke them on trunk by the working ones having a leading "e:" while the failing ones have a leading "/e/" in the path, so I guess we need the difference right in that sendchange call inside the upload diff (last thing to be invoked there) but we only rewrite it to be Windows-style right after this. The patch moves it to be before that, which actually matches what the original UnittestBuildFactory is doing.
![]() |
Assignee | |
Comment 6•14 years ago
|
||
(In reply to comment #4)
> I tried to solve this with
> http://hg.mozilla.org/build/buildbotcustom/rev/559708821cb9
This on the buildbot-0.7 branch and applied to our master has solved the original bug, but I'll only mark this fixed once this has reviews and landed on the default branch.
![]() |
||
Updated•14 years ago
|
Attachment #488793 -
Flags: review?(gozer) → review+
![]() |
Assignee | |
Comment 7•14 years ago
|
||
Thanks, checked in on default branch as http://hg.mozilla.org/build/buildbotcustom/rev/9e536c53028b
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Reporter | ||
Updated•14 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•