Dan Kenigsberg has submitted this change and it was merged. Change subject: iscsi.py: fix log message format ......................................................................
iscsi.py: fix log message format The logging methods accept the first argument as message, and rest arguments as format values and options. The original code wrongly splits a message into several arguments, so only the first part is recognized as message and the rest are recognized as values. Since there are no format specifiers in the actually recognized message, it will raise error when it finds there are format values. This patch utilizes the implicit string concat feature and fix this bug. Change-Id: I005f6343f2d8775dbf8406dffe04f6de93a81fce Signed-off-by: Zhou Zheng Sheng <[email protected]> --- M vdsm/storage/iscsi.py 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Yeela Kaplan: Verified; Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/9690 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I005f6343f2d8775dbf8406dffe04f6de93a81fce Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Yeela Kaplan <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
