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 <[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: Marcin Mirecki <[email protected]> Gerrit-Reviewer: Michal Skrivanek <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list -- [email protected] To unsubscribe send an email to [email protected]
