Change in ovirt-engine[master]: restapi: instance types need to have only the subset of temp...

tjelinek at redhat.com tjelinek at redhat.com
Fri Sep 25 11:40:02 UTC 2015


Tomas Jelinek has submitted this change and it was merged.

Change subject: restapi: instance types need to have only the subset of template fields
......................................................................


restapi: instance types need to have only the subset of template fields

InstanceTypeMapper was using the TemlpateMapper which was giving more fields to
instance type than it was supposed to.

Fixed by splitting out the mapCommon* from mapVmBase* and used this from
instance type mapper.

Change-Id: Ice3af5199d8f31aa175719743479c0aba89b4f2a
Bug-Url: https://bugzilla.redhat.com/1241279
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DisplayMapper.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/InstanceTypeMapper.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
4 files changed, 155 insertions(+), 91 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/46168
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ice3af5199d8f31aa175719743479c0aba89b4f2a
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernández
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list