Liron Aravot has submitted this change and it was merged.
Change subject: core: serialization/deserialization of SnapshotType
......................................................................
core: serialization/deserialization of SnapshotType
Because of the toString() implementation of SnapshotType (which is used
in returned messages) its deserialization is broken.
This patch fixes the serialization issue, as this code seems unneeded
to me I've removed it in change I8fb722dd15f0ccc2ac058765bfd6802a2a21134c.
This patch just fixes the serialization is it should be backported to the
3.6 branch and I'd like to minimize the changes and testing scope there.
Change-Id: Iee7192bd4e3bdf0b56ee57d4b25c5fcc17190be7
Bug-Url:
https://bugzilla.redhat.com/show_bug.cgi?id=1303840
Signed-off-by: Liron Aravot <laravot(a)redhat.com>
---
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Snapshot.java
1 file changed, 18 insertions(+), 0 deletions(-)
Approvals:
Allon Mureinik: Looks good to me, but someone else must approve
Moti Asayag: Looks good to me, approved
Liron Aravot: Verified; Passed CI tests
--
To view, visit
https://gerrit.ovirt.org/52995
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iee7192bd4e3bdf0b56ee57d4b25c5fcc17190be7
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Liron Aravot <laravot(a)redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>