Change in ovirt-engine[master]: core: fixing import storage domain to save additional NFS op...

oourfali at redhat.com oourfali at redhat.com
Thu Jul 26 13:06:39 UTC 2012


Oved Ourfali has submitted this change and it was merged.

Change subject: core: fixing import storage domain to save additional NFS options (#835961)
......................................................................


core: fixing import storage domain to save additional NFS options (#835961)

https://bugzilla.redhat.com/835961

This patch uses the AddStorageServerConnectionCommand in the import NFS
storage domain (same as when adding a new domain), instead of using
ConnectStorageServerVDSCommand, as it saves the server connection in the
database. Also, it puts the additional NFS options in the server
connection object as well, so that they will be persistent.

Change-Id: I87ac4a2b8a7f2083f2d5310cc9899514936d6111
Signed-off-by: Oved Ourfali <oourfali at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageListModel.java
1 file changed, 7 insertions(+), 3 deletions(-)

Approvals:
  Oved Ourfali: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I87ac4a2b8a7f2083f2d5310cc9899514936d6111
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Michael Kublin <mkublin at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>



More information about the Engine-commits mailing list