Dan Kenigsberg has submitted this change and it was merged.

Change subject: storageServer: Run glusterfs in its own cgroup
......................................................................


storageServer: Run glusterfs in its own cgroup

When mounting glusterfs volume, glusterfs command is started in the
vdsmd.service cgroup. When vdsmd exits, glusterfs dies and vms using
gluserfs volumes pause. Now we run glusterfs mount command using
systemd-run, so it run it its own cgroup, and last after vdsmd.service
exit.

In this example we have 2 glusterfs mounts:

 - vdsm.slice
   - vdsm-glusterfs.slice
     - run-22137.scope
       - 22180 /usr/bin/glusterfs ...
     - run-21649.scope
       - 21692 /usr/bin/glusterfs ...

The glusterfs processes may run few minutes after a volume is unmounted;
this happen also in the current code, running under vdsm cgroup.

Mounting should not leave child processes in the caller process tree.
This patch is a workaround until glusterfs fix this in the correct way.

Change-Id: I79714a9a6ecbde17c396eca6b17b9e3605fad688
Bug-Url: https://bugzilla.redhat.com/1201355
Signed-off-by: Sandro Bonazzola <sbona...@redhat.com>
Signed-off-by: Nir Soffer <nsof...@redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40240
Continuous-Integration: Jenkins CI
Reviewed-by: Federico Simoncelli <fsimo...@redhat.com>
---
M vdsm/storage/mount.py
M vdsm/storage/storageServer.py
2 files changed, 21 insertions(+), 2 deletions(-)

Approvals:
  Sandro Bonazzola: Verified; Looks good to me, but someone else must approve
  Nir Soffer: Verified
  Federico Simoncelli: Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/40240
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I79714a9a6ecbde17c396eca6b17b9e3605fad688
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimo...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbona...@redhat.com>
Gerrit-Reviewer: Vijay Bellur <vbel...@gmail.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to