Dan Kenigsberg has posted comments on this change. Change subject: networkTests: remove a broken approach to unmanaging devices by NetworkManager ......................................................................
Patch Set 2: Code-Review-1 (1 comment) -1 for visibility - I'd like to understand where and when our current tests break. http://gerrit.ovirt.org/#/c/37041/2//COMMIT_MSG Commit Message: Line 10: dnsmasq and dhclient on the respective sides) it was previously enough to set Line 11: an address on the client side to stop NetworkManager from running its own Line 12: dhclient on it. Line 13: Line 14: Now this approach is useless. On the server side, NetworkManager automatically what do you mean by "Now"? On which NetworkManager version does our approach work? Which version breaks it? Line 15: takes the veth up, so we cannot even assign an address to it, and the test is Line 16: skipped. Line 17: Line 18: Let's remove the broken code now. -- To view, visit http://gerrit.ovirt.org/37041 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idba14753bf9cd37ec1659a49ba7e13b9478f3913 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Ido Barkan <[email protected]> Gerrit-Reviewer: Ondřej Svoboda <[email protected]> Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
