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