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 zhshzhou@linux.vnet.ibm.com --- 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 zhshzhou@linux.vnet.ibm.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Yeela Kaplan ykaplan@redhat.com