summaryrefslogtreecommitdiff
path: root/Command/Log.hs
diff options
context:
space:
mode:
authorGravatar Joey Hess <joey@kitenet.net>2012-10-17 00:39:45 -0400
committerGravatar Joey Hess <joey@kitenet.net>2012-10-17 00:47:52 -0400
commitffde7cf5658c2f0224f0061eff4b9ef1e1daf3f4 (patch)
treeb8297bfbd91cdf89569081bfe54503bf31c25be9 /Command/Log.hs
parent640f0fea3e98b8b08c1bd995e14f9d37fa6f9d7d (diff)
better fix for zombie problem, which turns out to be a zombie ssh started by rsync
When rsyncProgress pipes rsync's stdout, this turns out to cause a ssh process started by rsync to be left behind as a zombie. I don't know why, but my recent zombie reaping cleanup was correct, it's just that this other zombie, that's not directly started by git-annex, was no longer reaped due to changes in the cleanup. Make rsyncProgress reap the zombie started by rsync, as a workaround. FWIW, the process tree looks like this. It seems like the rsync child is for some reason starting but not waiting on this extra ssh process. Ssh connection caching may be involved -- disabling it seemed to change the shape of the tree, but did not eliminate the zombie. 9378 pts/14 S+ 0:00 | \_ rsync -p --progress --inplace -4 -e 'ssh' '-S' ... 9379 pts/14 S+ 0:00 | | \_ ssh ... 9380 pts/14 S+ 0:00 | | \_ rsync -p --progress --inplace -4 -e 'ssh' '-S' ... 9381 pts/14 Z+ 0:00 | \_ [ssh] <defunct>
Diffstat (limited to 'Command/Log.hs')
-rw-r--r--Command/Log.hs2
1 files changed, 1 insertions, 1 deletions
diff --git a/Command/Log.hs b/Command/Log.hs
index c3ce67986..09d22b2ef 100644
--- a/Command/Log.hs
+++ b/Command/Log.hs
@@ -70,7 +70,7 @@ start :: M.Map UUID String -> TimeZone -> [CommandParam] -> Bool ->
start m zone os gource file (key, _) = do
showLog output =<< readLog <$> getLog key os
-- getLog produces a zombie; reap it
- liftIO Git.Command.reap
+ liftIO reapZombies
stop
where
output