Change in vdsm[master]: storage: streamline getting domain mountpoint

nsoffer at redhat.com nsoffer at redhat.com
Sun May 1 13:46:13 UTC 2016


Nir Soffer has submitted this change and it was merged.

Change subject: storage: streamline getting domain mountpoint
......................................................................


storage: streamline getting domain mountpoint

When checking if a domain's mountpoint is mounted, the code used to add
"/.." to the domain's path, counting on mount.isMounted() to normalize
the path.
This patch replaces this approach by simply using os.path.dirname.

Change-Id: Id1d8b8fb7851dd4f6b18e7804518227760a3e277
Signed-off-by: Idan Shaby <ishaby at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/56706
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Continuous-Integration: Jenkins CI
---
M vdsm/storage/glusterSD.py
M vdsm/storage/nfsSD.py
2 files changed, 8 insertions(+), 6 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Idan Shaby: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id1d8b8fb7851dd4f6b18e7804518227760a3e277
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Adam Litke <alitke at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the vdsm-patches mailing list