Change in ovirt-engine[master]: ui: wrong template displayed in edit Pool (#853727)

oourfali at redhat.com oourfali at redhat.com
Wed Sep 5 13:13:55 UTC 2012


Oved Ourfali has submitted this change and it was merged.

Change subject: ui: wrong template displayed in edit Pool (#853727)
......................................................................


ui: wrong template displayed in edit Pool (#853727)

https://bugzilla.redhat.com/853727

This patch does the following:
1. When editing a VM Pool, the wrong template is shown, thus the disks that
appear in the ResourceAllocation tab are also wrong, which causes
backend issues. This patch fixes that issue.
2. Disables the storage domain list in the resource
allocation tab when editing. Only viewing is allowed.
3. Fix audit log message when importing a VM template to contain the
target storage domain name.

Change-Id: I4e871596e1388cbda2697df72eb78540d59dad6b
Signed-off-by: Oved Ourfali <oourfali at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmTemplateCommand.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmModelBehaviorBase.java
4 files changed, 64 insertions(+), 10 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4e871596e1388cbda2697df72eb78540d59dad6b
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list