Change in ovirt-engine[master]: webadmin: update actions availability of VMs list with "excl...

From Tomas Jelinek <tjelinek@redhat.com>:
Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: update actions availability of VMs list with "exclusive" lock ...................................................................... webadmin: update actions availability of VMs list with "exclusive" lock Updating webAdmin actions availability of VMs in VmListModel to treat a VM in an "exclusive" memory lock and in "Down" status as if it is in an "ImageLocked" DB status and therefore block relevant actions for it. This is implemented by adding a new entity for overriding the getStatus() method of the VM entity, used only by VmListModel. This fix the problem that till now VMs in an "ImageLocked" status were checked and relevant actions were blocked for them in Webadmin as required, but VMs in an "exclusive" memory lock status were treated as if they are in "Down"/"Up"/Maintenance" etc status so evethough they were in the middle of a process like importing, relevant actions were not blocked for them as required. Change-Id: I51776fba35828589604ca44374a7574e91b4a20c Bug-Url: https://bugzilla.redhat.com/1364137 Signed-off-by: Sharon Gratch <sgratch@redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/VdcActionUtils.java A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmWithStatusForExclusiveLock.java M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java 5 files changed, 57 insertions(+), 17 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Sharon Gratch: Verified -- To view, visit https://gerrit.ovirt.org/71473 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I51776fba35828589604ca44374a7574e91b4a20c Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sharon Gratch <sgratch@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Gratch <sgratch@redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek@redhat.com> Gerrit-Reviewer: gerrit-hooks <automation@ovirt.org>
participants (1)
-
Code Review