From Dan Kenigsberg <dan...@redhat.com>: Dan Kenigsberg has submitted this change and it was merged. ( https://gerrit.ovirt.org/80908 )
Change subject: vdsm.tool.service: unbreak tool.service ...................................................................... vdsm.tool.service: unbreak tool.service In commit 53a2562 I have avoided execCmd splitting its output to lines, but I ignored a case where the split was actually used. I do not yet understand how did this passed my verification. Change-Id: I494871fc2834291143f7dc4c16ccc507ad5a1778 Signed-off-by: Dan Kenigsberg <dan...@redhat.com> --- M lib/vdsm/tool/service.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Piotr Kliczewski: Looks good to me, approved Jenkins CI: Passed CI tests Dan Kenigsberg: Verified -- To view, visit https://gerrit.ovirt.org/80908 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I494871fc2834291143f7dc4c16ccc507ad5a1778 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org