Closed
Bug 1415163
Opened 7 years ago
Closed 7 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•7 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•7 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•7 years ago
|
||
I guess I will have to switch to win64 and I will have to wait for bug 1413630.
Reporter | ||
Comment 4•7 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.
Comment 5•7 years ago
|
||
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•7 years ago
|
||
Tested on try, this is indeed fixed by bug 1404887.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•7 years ago
|
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
•