Dan Kenigsberg has submitted this change and it was merged. Change subject: Remove usage print on vdsm-tool.UsageError and use it on ExtraArgsError ......................................................................
Remove usage print on vdsm-tool.UsageError and use it on ExtraArgsError This print add usage explanation of vdsm-tool command on error. There is no need for such print Unless UsageError refers actually to the verb usage, but currently we use it for any usage error. On ExtraArgsError catch we add this usage print Change-Id: I6df376664ffcda66ea19f66e28edc0521566c091 Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com> Reviewed-on: http://gerrit.ovirt.org/28406 Reviewed-by: Dima Kuznetsov <dkuzn...@redhat.com> Reviewed-by: Dan Kenigsberg <dan...@redhat.com> --- M vdsm-tool/vdsm-tool 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Yaniv Bronhaim: Verified Dima Kuznetsov: Looks good to me, but someone else must approve Dan Kenigsberg: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/28406 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6df376664ffcda66ea19f66e28edc0521566c091 Gerrit-PatchSet: 4 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuzn...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches