Dan Kenigsberg has submitted this change and it was merged. Change subject: Changing vdsm-tool and vdsmd pre-tasks outputs to be more meaningful to user ......................................................................
Changing vdsm-tool and vdsmd pre-tasks outputs to be more meaningful to user Previously to this change the output was misleading. Users could not understand how to handle configuration issues properly. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1029812 Change-Id: If29f41081a70c0734cd78dab36f86c794a44b2d3 Signed-off-by: Yaniv Bronhaim <[email protected]> Reviewed-on: http://gerrit.ovirt.org/23253 Reviewed-by: Vered Volansky <[email protected]> Reviewed-by: Dan Kenigsberg <[email protected]> --- M init/vdsmd_init_common.sh.in M lib/vdsm/tool/configurator.py M lib/vdsm/tool/service.py 3 files changed, 26 insertions(+), 6 deletions(-) Approvals: Yaniv Bronhaim: Verified Dan Kenigsberg: Looks good to me, approved Vered Volansky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/23253 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If29f41081a70c0734cd78dab36f86c794a44b2d3 Gerrit-PatchSet: 4 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 _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
