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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org