Dan Kenigsberg has submitted this change and it was merged.

Change subject: caps: report if QEMU supports live snapshots
......................................................................


caps: report if QEMU supports live snapshots

Depending on QEMU version and configuration, live snapshotting
may be not supported. In that case, a request sent by engine
will of course fail.

Until version 1.2.3, libvirt did not expose any information about
the live snapshot support, so VDSM, and then engine, had no
option other than assume this was supported.

This patch adds detection and reporting of QEMU live snapshot
support, so engine can make informed choices like, for example,
disable the relevant fields in the UI.

Change-Id: I78dd51fc72f1b6d7eadb5c18d3b768f42d8ee32b
Bug-Url: https://bugzilla.redhat.com/1009100
Signed-off-by: Francesco Romani <[email protected]>
Reviewed-on: http://gerrit.ovirt.org/26149
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M debian/vdsm-tests.install
M tests/Makefile.am
M tests/capsTests.py
A tests/caps_libvirt_intel_i73770.out
A tests/caps_libvirt_intel_i73770_nosnap.out
M vdsm.spec.in
M vdsm/caps.py
M vdsm_api/vdsmapi-schema.json
8 files changed, 334 insertions(+), 5 deletions(-)

Approvals:
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I78dd51fc72f1b6d7eadb5c18d3b768f42d8ee32b
Gerrit-PatchSet: 11
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Leonardo Bianconi <[email protected]>
Gerrit-Reviewer: Michal Skrivanek <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Vitor de Lima <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to