Change in ovirt-engine[master]: rest: dont always use vm dynamic values

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Aug 27 08:18:26 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: rest: dont always use vm dynamic values
......................................................................


rest: dont always use vm dynamic values

Currently, when getting vm, some fields always show the dynamic values
(if exists) which contains information that is relevant for the
current/last run, for example, if vm has SPICE as display type, but on
run-once the user choose VNC, VNC will be shown while the vm is running.
another scenario, when user start the vm with SPICE and after that
change to VNC, SPICE is always shown, even if user explicitly request
the next_run info.

in this patch another option is added to the mapper to allow the caller
to decide if he wants the dynamic value or not, and it is used for the
next_run scenario. all other cases were not changed.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1128453
Change-Id: Iae643bd7f887171b389b71d31a36362ba0168709
Signed-off-by: Omer Frenkel <ofrenkel at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
2 files changed, 33 insertions(+), 16 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, approved
  Omer Frenkel: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iae643bd7f887171b389b71d31a36362ba0168709
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list