Dan Kenigsberg has posted comments on this change. Change subject: Add a libvirt migration hook to filter out min_guarantee element ......................................................................
Patch Set 1: Code-Review-1 (1 comment) http://gerrit.ovirt.org/#/c/32928/1/vdsm.spec.in File vdsm.spec.in: Line 699: install -Dm 0644 vdsm/vdsm-libvirt-access.pkla \ Line 700: %{buildroot}%{_polkitdir}/10-vdsm-libvirt-access.pkla Line 701: %endif Line 702: Line 703: # Install the libvirt hook for cleaning up the XML el6 does not need this, as it does not have the problematic libvirt-1.2.8. Since most of our users use el6, and since we have no experience with libvirt hooks yet, please avoid installing this hook on el6. Line 704: install -Dm 0755 vdsm/virt/libvirt-hook.sh \ Line 705: %{buildroot}%{_sysconfdir}/libvirt/hooks/qemu Line 706: Line 707: %check -- To view, visit http://gerrit.ovirt.org/32928 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I5f5e79037b755b257f9cac419887606f66705504 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msi...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Martin Sivák <msi...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches