Marcin Mirecki has posted comments on this change.
Change subject: virt: Migration plugins to enable skipping of special OVS processing for
nics
......................................................................
Patch Set 2:
The hook (vm_migrate_hook.py) was added as part of OVS support
(
https://gerrit.ovirt.org/#/c/59645/).
Whether that was a good or bad choice is out of the scope of this patch.
The problem with the original solution is that it breaks some hooks like vmfex or
openstack.
This patch is not bringing ovs support, but fixing an issue which ovs support has
introduced.
jsonrpc connections are not part of this patch, but part of the OVS support patch, and
doing anything about them is not the intent of this patch.
The purpose of this patch is only to allow to exclude some interfaces from the special OVS
interface handling.
--
To view, visit
https://gerrit.ovirt.org/64186
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia0fb056c4b4732505d567385546e78048c37d140
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh(a)redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com>
Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: No