Dan Kenigsberg has posted comments on this change.
Change subject: Unified network persistence [4.1/4.*] - Upgrade mechanism
......................................................................
Patch Set 17: Code-Review+2
(1 comment)
....................................................
File lib/vdsm/tool/upgrade.py
Line 81: def _editOtherLoggers(self, edit):
Line 82: """
Line 83: add/remove upgrade handler to/from all non-upgrade loggers
Line 84: """
Line 85: loggers = dict(logging.Logger.manager.loggerDict.items() +
Seems like an overkill to me, but let it be.
Line 86: [('root', logging.getLogger())])
Line 87: for name, logger in loggers.iteritems():
Line 88: if name != 'upgrade':
Line 89: for handler in logging.getLogger('upgrade').handlers:
--
To view, visit
http://gerrit.ovirt.org/17726
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iba3c9c34f03134c192db1c2add31084824e195d9
Gerrit-PatchSet: 17
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Assaf Muller <amuller(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap(a)redhat.com>
Gerrit-Reviewer: Assaf Muller <amuller(a)redhat.com>
Gerrit-Reviewer: Barak Azulay <bazulay(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce(a)redhat.com>
Gerrit-Reviewer: Giuseppe Vallarelli <gvallare(a)redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw(a)linux.vnet.ibm.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-HasComments: Yes