Change in vdsm[master]: ifcfg: Log unhandled exception for new Thread

danken at redhat.com danken at redhat.com
Mon Jan 13 09:35:21 UTC 2014


Dan Kenigsberg has posted comments on this change.

Change subject: ifcfg: Log unhandled exception for new Thread
......................................................................


Patch Set 1:

(1 comment)

....................................................
File vdsm/netconf/ifcfg.py
Line 737: 
Line 738: 
Line 739: def ifup(iface, async=False):
Line 740:     "Bring up an interface"
Line 741:     @utils.traceback()
I am not sure about the grand benefits of this: there's nothing in this function besides calling execCmd, which does its logging anyway. All I see is duplicated log noise.
Line 742:     def _ifup(netIf):
Line 743:         rc, out, err = utils.execCmd([constants.EXT_IFUP, netIf], raw=False)
Line 744: 
Line 745:         if rc != 0:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2ce44b4586e85438898fcdcd2d62d80813caa5ba
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list