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

tnisan at redhat.com tnisan at redhat.com
Tue Sep 29 09:42:07 UTC 2015


Tal Nisan 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
  Tal Nisan: Passed CI tests
  Juan Hernandez: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ice3af5199d8f31aa175719743479c0aba89b4f2a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez 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



More information about the Engine-commits mailing list