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(a)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(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com>