Change in vdsm[master]: net tests: IPv6 autoconf with RA server

phoracek at redhat.com phoracek at redhat.com
Tue Mar 15 15:21:48 UTC 2016


Petr Horáček has posted comments on this change.

Change subject: net tests: IPv6 autoconf with RA server
......................................................................


Patch Set 4:

(2 comments)

https://gerrit.ovirt.org/#/c/54668/4/tests/network/nettestlib.py
File tests/network/nettestlib.py:

Line 420: @contextmanager
Line 421: def wait_for_ipv6(iface, wait_for_scopes=None):
Line 422:     """Wait for iface to get their IPv6 addresses with netlink Monitor"""
Line 423:     if not wait_for_scopes:
Line 424:         wait_for_scopes = ['global', 'link']
mutables should not be default arguments?
Line 425:     try:
Line 426:         with monitor.Monitor(groups=('ipv6-ifaddr',), timeout=20) as mon:
Line 427:             yield
Line 428:             for event in mon:


Line 430: event.get('event') == 'new_addr' and
        :                    event.get('scope') in wait_for_scopes):
i think it's better to enclose all conditions, now it looks like there is some reason (for example OR) to explicitly enclose it.


-- 
To view, visit https://gerrit.ovirt.org/54668
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I7c153adf7d27db0151a330f55719622a8f730a9b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Edward Haas <edwardh at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Ondřej Svoboda <osvoboda at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list