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 <[email protected]>
Signed-off-by: Nir Soffer <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/40240
Continuous-Integration: Jenkins CI
Reviewed-by: Federico Simoncelli <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Vijay Bellur <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
