Change in ovirt-engine[master]: restapi: Remove Host#liveSnapshotSupport from V4 API

amureini at redhat.com amureini at redhat.com
Tue Apr 19 17:19:53 UTC 2016


Allon Mureinik has submitted this change and it was merged.

Change subject: restapi: Remove Host#liveSnapshotSupport from V4 API
......................................................................


restapi: Remove Host#liveSnapshotSupport from V4 API

The previous patch removed the liveSnapshotSupport property from the
backend VDS object and the related classes, but did not change the API
(other than hollowing it out).

This patch completes the job by consuming the updated
ovirt-engine-api-model artifact that removes this property from the new
V4 API.

In order to preserve backwards compatibility, the V3 API will continue
to report a `true` value, which is set in a hard-coded manner. Note that
even in the previous version this is a read-only property which could
not be updated (neither via the API nor by any other action via the
engine - the only way to affect this property is, theoretically, to
install/remove/update the qemu-kvm-[rhev] package on the host, but see
below). Additionally, any host that can participate in a 3.6 or newer
cluster will have this property as `true` anyway, as explained in the
previous patch.

Change-Id: I160d7ed8742128fdaac88bdc2228f4739139c5f5
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3HostInAdapter.java
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3HostOutAdapter.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
M pom.xml
4 files changed, 3 insertions(+), 10 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I160d7ed8742128fdaac88bdc2228f4739139c5f5
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini 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: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list