Adam Litke has submitted this change and it was merged.
Change subject: mount: Support NFS4 mounts with double slash
......................................................................
mount: Support NFS4 mounts with double slash
With NFS4, the kernel may present fs_spec using double slashes [1]:
noveria.wodel.wd://vmengine
Instead of the original (normalized) mount path:
noveria.wodel.wd:/vmengine
According to [2], this is unlikely to be fixed in the near future.
This patch normalizes fs_spec read from /proc/mounts, restoring
Mount.isMounted() with NFS4.
[1] http://lists.ovirt.org/pipermail/users/2016-July/041519.html
[2] http://www.spinics.net/lists/util-linux-ng/msg13079.html
Change-Id: Iee85efb6aaff5d3fe887d11d7c4a5045ae72ea57
Reported-By: wodel youchi <[email protected]>
Bug-Url: https://bugzilla.redhat.com/1361182
Signed-off-by: Nir Soffer <[email protected]>
Signed-off-by: Idan Shaby <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/61579
Continuous-Integration: Jenkins CI
Reviewed-by: Adam Litke <[email protected]>
---
M lib/vdsm/storage/fileUtils.py
M lib/vdsm/storage/mount.py
M tests/fileUtilTests.py
M tests/mountTests.py
4 files changed, 31 insertions(+), 8 deletions(-)
Approvals:
Adam Litke: Looks good to me, approved
Jenkins CI: Passed CI tests
Idan Shaby: Verified; Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/61579
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iee85efb6aaff5d3fe887d11d7c4a5045ae72ea57
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Freddy Rolland <[email protected]>
Gerrit-Reviewer: Idan Shaby <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/admin/lists/[email protected]
