[steeplechase] Mac leaves hanging diskimages-helper processes

RESOLVED FIXED

Status

Testing
New Frameworks
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: drno, Assigned: drno)

Tracking

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

3 years ago
All of the Mac's in the lab ran out of resources. It looks like they big amount of volumes mounted.

It looks to me like this line here:
https://github.com/mozilla/steeplechase/blob/master/steeplechase/runsteeplechase.py#L233
is mounting the Steeplechase volume twice. Running the umount command again makes sense, but mounting the steeplechase volume again after copying its content looks like copy and paste error to me.
(Assignee)

Comment 1

3 years ago
Syd, does the code above look OK to you, or might this be a bug?
Flags: needinfo?(spolk)
That code is just copying the .app package off of the DMG mount into a separate path. If you run a single test run on a mac from a fresh reboot, does the volume stay mounted? It's possible the unmount just isn't working properly for some reason.
(Assignee)

Comment 3

3 years ago
Sorry I clicked in wrong line of code. This is the line I'm worried about:
https://github.com/mozilla/steeplechase/blob/master/steeplechase/runsteeplechase.py#L235
(Assignee)

Comment 4

3 years ago
Also just realized that this line:
https://github.com/mozilla/steeplechase/blob/master/steeplechase/runsteeplechase.py#L237
doesn't have any logging.
I don't think that the exact line makes any difference there. :) line 233 is just assigning the command to the cmd variable, 234 logs it, and 235 executes it. The command in 237 is logged in the previous line, isn't it?
(Assignee)

Comment 6

3 years ago
You are right Ted, I overlooked that it only logs what it is about to execute and not the result.
And I also overlooked that cmd gets redefined.

Never the less we have an issue on Mac. Yesterday I went into all the Mac's we have as part of our setup and manually killed all the hanging diskimage-helper processes. Since then the result now looks like this:

pf-mac10-10-01:~ mozilla$ ps auxwwww | grep diskim | wc -l
      47

The processes itself looks like this:

mozilla          9926   0.0  0.2  2503656   3656   ??  Ss    5:03PM   0:26.34 /System/Library/PrivateFrameworks/DiskImages.framework/Resources/diskimages-helper -uuid 42EC1FE2-4755-4DFC-A1CF-3B503C2B3B2F -post-exec 4
(Assignee)

Updated

3 years ago
Summary: [steeplechase] Mac leaves mounted volumes → [steeplechase] Mac leaves hanging diskimages-helper processes
Are the disk images still mounted if you look at the output of `mount`, or is it just that these processes hang around?
(Assignee)

Comment 8

3 years ago
No there is nothing mounted. But just these processes hanging around. But the problem with the hanging processes is that if enough of them hang around you can't even open a Terminal any more.

BTW you can only remove these hanging processes with SIGKILL, they don't die on SIGTERM.

I just watched one machine after manually cleaning the hanging processes before the test run:

During the test run these two processes get created:

pf-mac10-10-01:~ mozilla$ ps auxww | grep diskimage
mozilla         11912 100.8  0.6  2501568  13248   ??  R    11:05AM   0:08.56 /System/Library/PrivateFrameworks/DiskImages.framework/Resources/diskimages-helper -uuid EC7D4F92-1DBE-4F1D-B4B0-0D27159B348D -post-exec 4
mozilla         11916   0.0  0.0  2442588    436 s000  R+   11:06AM   0:00.00 grep diskimage
mozilla         11911   0.0  0.1  2442584   2368   ??  S    11:05AM   0:00.01 /System/Library/PrivateFrameworks/DiskImages.framework/Resources/diskimages-helper -uuid EC7D4F92-1DBE-4F1D-B4B0-0D27159B348D

Then the disk is mounted, stuff gets copied, disk gets unmounted, tests executes and at the end we still have this:

pf-mac10-10-01:~ mozilla$ ps auxww | grep diskimage
mozilla         11950   0.4  0.0  2432772    632 s000  S+   11:06AM   0:00.00 grep diskimage
mozilla         11912   0.0  0.8  2503488  17068   ??  Ss   11:05AM   0:25.64 /System/Library/PrivateFrameworks/DiskImages.framework/Resources/diskimages-helper -uuid EC7D4F92-1DBE-4F1D-B4B0-0D27159B348D -post-exec 4
(Assignee)

Comment 9

3 years ago
Aha, apparently we gets lots of still attached disks:

http://stackoverflow.com/questions/4046019/error-when-detaching-volume-using-hdiutil-on-os-x
I was just about to post that same link, looks like we just need to switch "unmount" for "detach" to solve this.
(Assignee)

Updated

3 years ago
Assignee: nobody → drno
(Assignee)

Comment 11

3 years ago
Working on it...
(Assignee)

Comment 12

3 years ago
I created a Pull request https://github.com/mozilla/steeplechase/pull/4
Ted can you review and merge please?
Flags: needinfo?(spolk)

Comment 13

3 years ago
I merged the pull request. hdiutil detach is the correct thing to do.

Updated

3 years ago
Status: NEW → ASSIGNED

Updated

3 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
(Assignee)

Comment 14

3 years ago
Verified. Works. No hanging processes any more.
You need to log in before you can comment on or make changes to this bug.