Dan Kenigsberg has submitted this change and it was merged.
Change subject: virt: ovs: fix migrations without the ovs hook
......................................................................
virt: ovs: fix migrations without the ovs hook
Since commit 50de041 the libvirt migration hook
that Vdsm always installs started to depend on the
ovs_migrate.py helper, which is in turn provided
by the optional ovs hook.
So, without the hook installed a given host, any incoming
migration with fail with
libvirtd[1179]: internal error: Child process [...]
/etc/libvirt/hooks/qemu a_c7_0 migrate begin -)
unexpected exit status 127:
/etc/libvirt/hooks/qemu: line 20:
/usr/libexec/vdsm/ovs_migrate.py: No such file or directory
(reformatted for clarity and to fit in 78 columns).
This patch fixes this, making incoming migrations work again even
without the ovs hook installed.
Change-Id: I70fc10782255a71855de63fe7dc7f128d76e6306
Signed-off-by: Francesco Romani <[email protected]>
Reviewed-on: https://gerrit.ovirt.org/56034
Continuous-Integration: Jenkins CI
Reviewed-by: Edward Haas <[email protected]>
Reviewed-by: Dan Kenigsberg <[email protected]>
---
M vdsm/virt/libvirt-hook.sh
1 file changed, 5 insertions(+), 1 deletion(-)
Approvals:
Jenkins CI: Passed CI tests
Dan Kenigsberg: Looks good to me, approved
Francesco Romani: Verified
Edward Haas: Looks good to me, but someone else must approve
--
To view, visit https://gerrit.ovirt.org/56034
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I70fc10782255a71855de63fe7dc7f128d76e6306
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Edward Haas <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: gerrit-hooks <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
