Edward Haas has posted comments on this change. Change subject: ovs: add network with dhcp configuration ......................................................................
Patch Set 9: Code-Review-1 (3 comments) https://gerrit.ovirt.org/#/c/59039/8/tests/network/func_dhclient_test.py File tests/network/func_dhclient_test.py: Line 65: NETWORK_NAME, netcreate[NETWORK_NAME]) Line 66: finally: Line 67: if bridged: Line 68: delete_dhclient_leases( Line 69: NETWORK_NAME, dhcpv4=True, dhcpv6=False) > dhclient keeps leases for 2 minuts, we want to do every setup as a first on I'm not convinced, sounds like a leftover from the time we checked the leases to determine dhcp status. If you want to keep it, then lets move it to the dnsmasq_run context. Line 70: else: Line 71: delete_dhclient_leases( Line 72: client, dhcpv4=True, dhcpv6=False) Line 73: https://gerrit.ovirt.org/#/c/59039/9/tests/network/func_dhclient_test.py File tests/network/func_dhclient_test.py: PS9, Line 35: _LEN Not needed. bitmask is the mask in number of bits. https://gerrit.ovirt.org/#/c/59039/9/tests/network/netfunctestlib.py File tests/network/netfunctestlib.py: Line 243: self.update_running_config() Line 244: self.assertNotIn(bond, self.running_config.bonds) Line 245: Line 246: def assertNetworkIp(self, net, attrs): Line 247: if ('ipaddr' not in attrs and attrs.get('bootproto') != 'dhcp' and You better also take care of dhcpv6 and ipv6autoconf as well here. Line 248: 'ipv6addr' not in attrs): Line 249: return Line 250: Line 251: network_netinfo = self.netinfo.networks[net] -- To view, visit https://gerrit.ovirt.org/59039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I792713f27f41faf5e2ffffd4de02c51ad8d0ee02 Gerrit-PatchSet: 9 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Petr Horáček <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Edward Haas <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Petr Horáček <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
