Closed
Bug 1415163
Opened 8 years ago
Closed 8 years ago
mozinstall fails with InstallError on test machines with Windows code coverage build
Categories
(Testing :: Mozbase, defect)
Tracking
(firefox58 fixed)
RESOLVED
FIXED
mozilla58
| Tracking | Status | |
|---|---|---|
| firefox58 | --- | fixed |
People
(Reporter: marco, Assigned: froydnj)
References
Details
Attachments
(1 file)
|
9.75 KB,
application/x-xz
|
Details |
https://treeherder.mozilla.org/#/jobs?repo=try&revision=d42fe1917bf50913621793a96762fb7a98ff361a
Any idea what could be the cause of the failure? Could you help me debug it?
Flags: needinfo?(hskupin)
Flags: needinfo?(ahalberstadt)
Comment 1•8 years ago
|
||
Mozinstall is trying to be too clever for its own good in the exception handling and it's clobbering the original exception type. Try adding a 'raise' here to prevent that:
https://dxr.mozilla.org/mozilla-central/source/testing/mozbase/mozinstall/mozinstall/mozinstall.py#138
That should at least make the error a bit more useful. Is it possible the zip file it's trying to extract is malformed or empty or something?
Flags: needinfo?(ahalberstadt)
| Reporter | ||
Comment 2•8 years ago
|
||
I've completely removed the exception handling part, the exception is a "MemoryError": https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b3f2d42b0fc453d8d7a91944831458753198a21&selectedJob=142824908.
| Reporter | ||
Comment 3•8 years ago
|
||
I guess I will have to switch to win64 and I will have to wait for bug 1413630.
| Reporter | ||
Comment 4•8 years ago
|
||
Or maybe we can replace https://dxr.mozilla.org/mozilla-central/rev/2535bad09d720e71a982f3f70dd6925f66ab8ec7/testing/mozbase/mozfile/mozfile/mozfile.py#73 to avoid reading the contents of the zip entry in memory and extract it directly.
That's exactly what we work on bug 1404887. So this issue might be fixed by the other bug?
Depends on: 1404887
Flags: needinfo?(hskupin)
| Reporter | ||
Comment 6•8 years ago
|
||
Tested on try, this is indeed fixed by bug 1404887.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Assignee: nobody → nfroyd
status-firefox58:
--- → fixed
No longer depends on: 1413630
Target Milestone: --- → mozilla58
You need to log in
before you can comment on or make changes to this bug.
Description
•