Change in ovirt-engine[ovirt-engine-3.5]: core: assume live snapshot supported as default

tnisan at redhat.com tnisan at redhat.com
Mon Nov 3 11:58:59 UTC 2014


Tal Nisan has submitted this change and it was merged.

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.

Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1159211
Related-To: http://bugzilla.redhat.com/1149460
Related-To: http://bugzilla.redhat.com/1158974
Change-Id: I4660560e08f606e2b16a580ef60618abfad49244
Signed-off-by: Francesco Romani <fromani at redhat.com>
---
M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java
1 file changed, 2 insertions(+), 0 deletions(-)

Approvals:
  Francesco Romani: Verified
  Omer Frenkel: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/34742
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I4660560e08f606e2b16a580ef60618abfad49244
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list