Change in vdsm[master]: network: ifcfg: synchronous ifup

phoracek at redhat.com phoracek at redhat.com
Wed Jan 28 12:59:46 UTC 2015


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

Change subject: network: ifcfg: synchronous ifup
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/37366/1/vdsm/network/configurators/ifcfg.py
File vdsm/network/configurators/ifcfg.py:

Line 857: 
Line 858: 
Line 859: @contextmanager
Line 860: def _event_sniffer(expected_event, timeout=10):
Line 861:     mon = monitor.Monitor(groups=('link', 'ipv4-ifaddr', 'ipv6-ifaddr'),
use context manager
Line 862:                           timeout=timeout)
Line 863:     mon.start()
Line 864:     try:
Line 865:         yield


-- 
To view, visit http://gerrit.ovirt.org/37366
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0c90a556f5fc52c1b8e675986ac39b6032ca0197
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Petr Horáček <phoracek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list