Change in ovirt-engine[master]: webadmin: validation for "Use Host CPU" setting

lhornyak at redhat.com lhornyak at redhat.com
Wed Feb 13 13:13:03 UTC 2013


Laszlo Hornyak has submitted this change and it was merged.

Change subject: webadmin: validation for "Use Host CPU" setting
......................................................................


webadmin: validation for "Use Host CPU" setting

To enable CPU Host, one of the two no-migrate flags for a VM (run on
selected host, or allow migration only upon admin request) must be
enabled.  If CPU Host was disabled along with the no-migrate flag in
the same save action, the new CPU Host value was not propagated to the
VM object which caused a validation error.

This patch propagates the CPU Host value to the VM object, allowing
the CPU Host flag and no-migrate flag to be disabled in the same edit.

Change-Id: I786a725f6ff20e64d2d094b4c8c183c907978229
Bug-Url: https://bugzilla.redhat.com/905843
Signed-off-by: Greg Padgett <gpadgett at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/VmListModel.java
1 file changed, 4 insertions(+), 0 deletions(-)

Approvals:
  Laszlo Hornyak: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I786a725f6ff20e64d2d094b4c8c183c907978229
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett at redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak at redhat.com>



More information about the Engine-commits mailing list