Change in ovirt-engine[master]: webadmin: Pending VM changes dialog fix

mbetak at redhat.com mbetak at redhat.com
Tue Aug 30 08:35:46 UTC 2016


Martin Betak has submitted this change and it was merged.

Change subject: webadmin: Pending VM changes dialog fix
......................................................................


webadmin: Pending VM changes dialog fix

Dialog is shown even if there are only changes that "can be applied
immediately", i.e. CPU or memory.

Previously there had to be at least one change "that require VM restart"
for the dialog to me shown.

Change-Id: Ib730030c766450011d216f379c87fd6f15136006
Bug-Url: https://bugzilla.redhat.com/1351208
Signed-off-by: Jakub Niedermertl <jniederm at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmNextRunConfigurationWidget.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
2 files changed, 8 insertions(+), 7 deletions(-)

Approvals:
  Tomas Jelinek: Looks good to me, approved
  Jakub Niedermertl: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/62613
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib730030c766450011d216f379c87fd6f15136006
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jakub Niedermertl <jniederm at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Betak <mbetak at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list