Change in ovirt-engine[master]: core: Fix removal of Gluster storage on Posix FS (#857632)

tnisan at redhat.com tnisan at redhat.com
Mon Oct 15 17:34:18 UTC 2012


Tal Nisan has submitted this change and it was merged.

Change subject: core: Fix removal of Gluster storage on Posix FS (#857632)
......................................................................


core: Fix removal of Gluster storage on Posix FS (#857632)

When a domain is detached from a storage pool, the advanced options
did not pass to the ConnectStorageServer VDS command and thus in case
of Gluster storage the vfstype option did not pass to the host and it
doesn't try to mount it as a Gluster storage, this was fixed by including
the vfstype option even if the domain is detached

Change-Id: I5de20f13a1c03f230298f92bfe19643be8900ac0
Bug-Url: https://bugzilla.redhat.com/857632
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/ConnectStorageServerVDSCommand.java
1 file changed, 6 insertions(+), 3 deletions(-)

Approvals:
  Tal Nisan: Verified
  Daniel Erez: Looks good to me, approved
  Allon Mureinik: Looks good to me, but someone else must approve


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5de20f13a1c03f230298f92bfe19643be8900ac0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>



More information about the Engine-commits mailing list