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