Michal Skrivanek has posted comments on this change.
Change subject: virt: Migration plugins to enable skipping of special OVS processing for
nics
......................................................................
Patch Set 2:
It is out of scope of this patch. But the series of patches introduced on top of the
original patch is alarming and suggesting that indeed the original patch was not right.
Then, I'd rather prefer to declare OVS as not working and go back to design board.
Continuing with trying to patch and fix it seems to lead to more and more troubles and
unsustainable code.
as francesco says, if it unblocks something critical, so be it, merge it if you want to,
but I urge you (you, as a network team) to revisit the OVS feature, its constraints and
desired behavior, and how does it fit (or not) libvirt.
--
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