Dan Kenigsberg has posted comments on this change.
Change subject: Make API.py conform to pep8 ......................................................................
Patch Set 3: Looks good to me, but someone else must approve
(1 inline comment)
.................................................... File vdsm/API.py Line 1117: logging.getLogger('clientIF.setLogLevel').info('Setting loglevel' + \ note that within parenthesis, backslash is not really required or helpful.
-- To view, visit http://gerrit.ovirt.org/3561 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: I9549186e537ba23994891e27b58be3d6d3770f40 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Adam Litke agl@us.ibm.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Lei Li lilei@linux.vnet.ibm.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com