Dan Kenigsberg has posted comments on this change.
Change subject: Fix logging formatting in resourceManager
......................................................................
Patch Set 2: I would prefer that you didn't submit this
(1 inline comment)
mainly wondering - why? just a matter of taste? (which I share)
....................................................
File vdsm/storage/resourceManager.py
Line 571: self._log.debug("Request '%s' was
canceled, Ignoring it.", str(nextRequest))
is there anything wrong in using the generic % format operator?
(there is something superfluous and wrong in using str(), which croaks on unicode while
'%s' handles it just fine)
--
To view, visit
http://gerrit.ovirt.org/531
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I08b530e8fbf66b9b8326cf1733df6a3e4f23ef1a
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>