Change in vdsm[master]: gluster: raise proper execption if the given mount point is ...

danken at redhat.com danken at redhat.com
Fri May 15 11:02:49 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: gluster: raise proper execption if the given mount point is in use.
......................................................................


gluster: raise proper execption if the given mount point is in use.

Currently if an already used mount point is given for the brick
create function, it does all the lvm tasks and returns Unexpected
Exception when it tries to mout the device at the end of brick
create scenario. This patch will check the availablity of the given
mount point and returns an appropriate error message.

Change-Id: Ic12acd2e4bf6a88936af582bb17d50d72cdfb84f
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1209475
Signed-off-by: Timothy Asir Jeyasingh <tjeyasin at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/40892
Continuous-Integration: Jenkins CI
Reviewed-by: Bala.FA <barumuga at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/gluster/exception.py
M vdsm/gluster/storagedev.py
2 files changed, 14 insertions(+), 0 deletions(-)

Approvals:
  Timothy Asir: Verified
  Bala.FA: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic12acd2e4bf6a88936af582bb17d50d72cdfb84f
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: Bala.FA <barumuga at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Darshan N <dnarayan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ramesh N <rnachimu at redhat.com>
Gerrit-Reviewer: Sahina Bose <sabose at redhat.com>
Gerrit-Reviewer: Timothy Asir <tjeyasin at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list