Tomas Jelinek has uploaded a new change for review. Change subject: engine: Edit Pool shows different RAM than defined (#833829) ......................................................................
engine: Edit Pool shows different RAM than defined (#833829) https://bugzilla.redhat.com/833829 Have a template X with 256 MB memory, than: Pools->New->select template X->change RAM to 512 MB->save Than select the newly created pool and Edit. The memory will be 256 MB. The problem was, that both the (New/Existing)PoolModelBehavior classes let the filling of the window model to the parent class (PoolModelBehaviorBase) which used the template. It is correct for the NewPoolModelBehavior but not for the ExistingPoolModelBehavior. Fixed by generalizing the logic of setting up the window model to take VmBase instead of VmTemplate and calling this from the specific classes (New/Existing)PoolModelBehavior with the correct parameter. This patch also adds the "num_of_cpus" to ObjectIdentityChecker instance in the VmHandler class. This enables the cpu editing in the VM popup view. Change-Id: Idaa1a92f7fe85f9c604da8916be5672d8d5b1e8b Signed-off-by: Tomas Jelinek <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmBase.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmStatic.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingPoolModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/NewPoolModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java 7 files changed, 84 insertions(+), 47 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/38/6638/1 -- To view, visit http://gerrit.ovirt.org/6638 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Idaa1a92f7fe85f9c604da8916be5672d8d5b1e8b Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tomas Jelinek <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
