Francesco Romani has uploaded a new change for review. Change subject: core: assume live snapshot supported as default ......................................................................
core: assume live snapshot supported as default VDSM will report if underlying QEMU reports live snapshot or not. This information is provided through libvirt. If libvirt, or VDSM, no longer report this capability, Engine get stuck with the old value and it is no longer updated. There is an unlucky case on CentOS and RHEL 6.6 on which we can hit this condition. To fix that, this patch assumes live snapshot is supported if we don't have word from VDSM. Change-Id: I4660560e08f606e2b16a580ef60618abfad49244 Signed-off-by: Francesco Romani <[email protected]> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 2 insertions(+), 0 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/73/34673/1 diff --git a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java index 07a156c..ed5d09e 100644 --- a/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java +++ b/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java @@ -511,6 +511,8 @@ if (xmlRpcStruct.containsKey(VdsProperties.liveSnapshotSupport)) { vds.setLiveSnapshotSupport(AssignBoolValue(xmlRpcStruct, VdsProperties.liveSnapshotSupport)); + } else { + vds.setLiveSnapshotSupport(true); // for backward compatibility's sake } if (xmlRpcStruct.containsKey(VdsProperties.liveMergeSupport)) { vds.setLiveMergeSupport(AssignBoolValue(xmlRpcStruct, VdsProperties.liveMergeSupport)); -- To view, visit http://gerrit.ovirt.org/34673 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I4660560e08f606e2b16a580ef60618abfad49244 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Francesco Romani <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
