Yaniv Bronhaim has submitted this change and it was merged.

Change subject: fileSD: include gluster in getMountsList
......................................................................


fileSD: include gluster in getMountsList

It is not exactly clear what previous change broke scanDomains but
looking at the code it seems that gluster was never explicitly handled.

This patch adds the explicit support for gluster in scanDomains fixing
the problem of missing storage domain links in the pool directory.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1116585
Change-Id: I81600a68bbea99c1079bf153cf2c105c6fe73f70
Signed-off-by: Federico Simoncelli <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/30934
Reviewed-by: Dan Kenigsberg <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/31158
Reviewed-by: Nir Soffer <[email protected]>
Reviewed-by: Allon Mureinik <[email protected]>
Reviewed-by: Yaniv Bronhaim <[email protected]>
---
M vdsm/storage/fileSD.py
1 file changed, 18 insertions(+), 6 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Yaniv Bronhaim: Looks good to me, approved
  Federico Simoncelli: Verified
  Allon Mureinik: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/31158
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I81600a68bbea99c1079bf153cf2c105c6fe73f70
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Nir Soffer <[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

Reply via email to