Change in ovirt-engine[ovirt-engine-3.5]: core: changed the list of fields which can not be overridden

tnisan at redhat.com tnisan at redhat.com
Sun Nov 16 10:51:26 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: core: changed the list of fields which can not be overridden
......................................................................


core: changed the list of fields which can not be overridden

Some of the fields on the instance types are prohibited to be overridden
because in such a case the VM would not be considered to be based on that
instance type anymore.

This patch reduces the num of fields which can not be overridden to contain
only the really most important ones. This avoids the situation that the user will
have to have instance types like: "big with soundcard", "big with smardcard and
soundcard" etc.

The list is now significantly smaller, contains only:
- memory size
- num of sockets/cores per socket
- HA
- migration model/downtime
- priority
- balloon/min allocated memory

Change-Id: I6142a240d60d54634361d3e6427b2a56fbd01362
Bug-Url: https://bugzilla.redhat.com/1149663
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VmManagementCommandBase.java
1 file changed, 14 insertions(+), 23 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Omer Frenkel: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I6142a240d60d54634361d3e6427b2a56fbd01362
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list