Change in ovirt-engine[master]: webadmin: Correct 64-bit VM memory limit validation

tjelinek at redhat.com tjelinek at redhat.com
Tue Apr 23 14:01:22 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: webadmin: Correct 64-bit VM memory limit validation
......................................................................


webadmin: Correct 64-bit VM memory limit validation

The UnitVmModel.UpdateMaximalVmMemSize method wasn't called on
Cluster_SelectedItemChanged, so the validation used the initial value of
262144 MB which is obsolete.

Also changed the default value for ConfigValues.VM64BitMaxMemorySizeInMB from
262144 to more up-to-date 524288 (supported since 3.0).

Change-Id: I16562b554a133e9c753a859d6f3f5a348eaa72cc
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=954021
Signed-off-by: Martin Betak <mbetak at redhat.com>
---
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
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/UnitVmModel.java
3 files changed, 6 insertions(+), 3 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I16562b554a133e9c753a859d6f3f5a348eaa72cc
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Martin Beták <mbetak at redhat.com>
Gerrit-Reviewer: Martin Beták <mbetak at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list