Francesco Romani has posted comments on this change.
Change subject: virt: Migration plugins to enable skipping of special OVS processing for
nics
......................................................................
Patch Set 2:
(1 comment)
https://gerrit.ovirt.org/#/c/64186/2//COMMIT_MSG
Commit Message:
PS2, Line 11: plugins
This should probably replaced by:
OK for plugins.
About my ACK, for short term I think we don't have other option to let everything
work, do we?
So ok, let's do this to unbreak things.
But again, this path is not sustainable in the long term, we need to discuss if we need
fixes/RFEs in the vdsm core, or even at libvirt level.
Example of what I mean: do we need to teach libvirt about ovs?
Do we need to ask libvirt to run multiple hooks, and not just one?
--
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: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes