On Mon, Mar 6, 2017 at 11:18 AM, Ryan Harper <[email protected]> wrote:
> On Mon, Mar 6, 2017 at 11:13 AM, Mathieu Trudel-Lapierre < > [email protected]> wrote: > > > Review: Needs Fixing > > > > I wasn't able to successfully verify that this works on both NM and > > networkd. Could you add integration tests too? > > > > I tested networkd only; I'll look at the integration tests. > I'm struggling with getting the tests to work reliably; If anyone of the tests fails, the system ends up in a bad state (modules previously loaded fail to a load a second time); veth devices stick around, and data is written into /run but not cleaned up between test-case runs. The tests don't run with via nose; but rather calling python directly and only from within the tests directory. Finally the test invokes 'netplan' which references whatever netplan is installed on the system rather than the netplan from the source tree. I didn't verify NM side as I didn't add any changes for NM side; > > > -- > > https://code.launchpad.net/~raharper/netplan/+git/netplan/+merge/318545 > > You are the owner of ~raharper/netplan:fix-mtu. > > > > -- > https://code.launchpad.net/~raharper/netplan/+git/netplan/+merge/318545 > You are the owner of ~raharper/netplan:fix-mtu. > -- https://code.launchpad.net/~raharper/netplan/+git/netplan/+merge/318545 Your team Developers of netplan is subscribed to branch netplan:master. -- Mailing list: https://launchpad.net/~netplan-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~netplan-developers More help : https://help.launchpad.net/ListHelp
