Change in ovirt-engine[ovirt-engine-3.4]: webadmin: Fix allocation policy and storage domain when edit...
iheim at redhat.com
iheim at redhat.com
Wed Feb 19 15:49:04 UTC 2014
Itamar Heim has submitted this change and it was merged.
Change subject: webadmin: Fix allocation policy and storage domain when editing VM disk
......................................................................
webadmin: Fix allocation policy and storage domain when editing VM disk
Edit Vm disk window always displayed disk allocation policy based on the
storage type instead of displaying the actual allocation policy of that
disk.
In case of the storage domain, first domain from the list of available
storage domains was displayed instead of the storage domain of that disk.
Change-Id: Ifae8052b4e37a7ad7f39eb38081268a7b6bd1a43
Bug-Url: https://bugzilla.redhat.com/1057622
Signed-off-by: Sergey Gotliv <sgotliv at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/AbstractDiskModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/EditDiskModel.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewDiskModel.java
3 files changed, 20 insertions(+), 7 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Sergey Gotliv: Verified
--
To view, visit http://gerrit.ovirt.org/24686
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ifae8052b4e37a7ad7f39eb38081268a7b6bd1a43
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgotliv at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list