811830 - DMError: Attempted to push a file (libfreebl3.so) to a directory (/system/b2g/libfreebl3.so) in install_gecko

Status: RESOLVED WONTFIX

(Testing :: Mozbase, defect)

Description

[Andrew Halberstadt [:ahal]]

See: https://tbpl.mozilla.org/php/getParsedLog.php?id=17033701&tree=Mozilla-Inbound

I'm not sure if this is an error with how we are setting things up in emulator.py, or if it's a devicemanager pushFile()/dirExists() error. Filing in testing/marionette for now.

Comment 1

[Andrew Halberstadt [:ahal]]

<wlach> ahal: I think we might have to call communicate() in isDirectory for it to work properly in all cases. we might have a race condition between stdout having data and us reading from it
<wlach> ahal: yeah I think it's more of a mozbase bug
<wlach> bleh, if this is true this is probably far from the only case in dmADB where this runCmdAs stuff is causing problems

Comment 2

[William Lachance (:wlach)]

So I guess the question is what we want to do here. It looks like we're quite frequently calling read() on the Popen class returned by _runCmd frequently, without checking whether the process has actually terminated.

I think the proper solution is to make _runCmd call communicate() and return the stdout/stderr directly to the caller (instead of returning a subclass). In addition to modifying deviceManagerADB, we'd also need to modify anything that uses _runCmd directly:

http://mxr.mozilla.org/mozilla-central/search?string=._runCmd&find=&findi=&filter=^[^\0]*%24&hitlimit=&tree=mozilla-central

If we're going to make these changes, it may also be worthwhile to modify some uses of _runCmd (both inside and outside dmADB, especially outside) to use shellCheckOutput, which has better error checking.

I don't really have the time to do this right now, but will probably will be able to tackle this at some point in the next few weeks. If anyone feels like fixing this in the interim, go ahead. :) In the mean time, this is probably a very infrequent problem.

Comment 3

[William Lachance (:wlach)]

Thinking about this a bit more, I think the right solution is probably to replace _runCmd and _runCmdAs with functions that are less difficult to use.

Comment 4

[Andrew Halberstadt [:ahal]]

For some reason B2G xpcshell tests are consistently hitting this error only on the mozilla-b2g18 branch :(

https://tbpl.mozilla.org/php/getParsedLog.php?id=17903872&tree=Mozilla-B2g18&full=1

This is probably top priority for me now.

Comment 5

[Andrew Halberstadt [:ahal]]

Actually, upon further investigation, this is unrelated to this bug. It looks like the version of remotexpcshelltests.py on the beta/b2g18 branch just needs to be updated.

Comment 6

[Phil Ringnalda (:philor)]

https://tbpl.mozilla.org/php/getParsedLog.php?id=20831895&tree=Mozilla-Inbound