Yaniv Bronhaim has posted comments on this change. Change subject: Changing vdsm-tool and vdsmd pre-tasks outputs to be more meaningful to user ......................................................................
Patch Set 1: (1 comment) thanks. will fix the typos .. (that's the meaning of the patch, so for me it deserves -1 with typos) http://gerrit.ovirt.org/#/c/23253/1/lib/vdsm/tool/configurator.py File lib/vdsm/tool/configurator.py: Line 247: Line 248: if not ret: Line 249: msg = \ Line 250: """ Line 251: > Is there a redundant empty line here or is it just gerrit? its in purpose to split the above errors from my very well explained comment afterwards Line 252: One of the modules is not configured to VDSM. Line 253: To configure the module use the following: Line 254: 'vdsm-tool configure [module_name]'. Line 255: -- To view, visit http://gerrit.ovirt.org/23253 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: If29f41081a70c0734cd78dab36f86c794a44b2d3 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
