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)

Version 3
defect
Not set
normal

Tracking

(firefox58 fixed)

RESOLVED FIXED
mozilla58
Tracking Status
firefox58 --- fixed

People

(Reporter: marco, Assigned: froydnj)

References

Details

Attachments

(1 file)

Attached file log_raw.tar.xz
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)
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)
I've completely removed the exception handling part, the exception is a "MemoryError": https://treeherder.mozilla.org/#/jobs?repo=try&revision=6b3f2d42b0fc453d8d7a91944831458753198a21&selectedJob=142824908.
I guess I will have to switch to win64 and I will have to wait for bug 1413630.
Depends on: 1413630
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)
Tested on try, this is indeed fixed by bug 1404887.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Assignee: nobody → nfroyd
No longer depends on: 1413630
Target Milestone: --- → mozilla58
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: