Yaniv Bronhaim has uploaded a new change for review.
Change subject: Remove redundant print
......................................................................
Remove redundant print
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.
Change-Id: I6df376664ffcda66ea19f66e28edc0521566c091
Signed-off-by: Yaniv Bronhaim <ybron...@redhat.com>
---
M vdsm-tool/vdsm-tool
1 file changed, 0 insertions(+), 1 deletion(-)
git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/06/28406/1
diff --git a/vdsm-tool/vdsm-tool b/vdsm-tool/vdsm-tool
index 21e378f..365ff3c 100755
--- a/vdsm-tool/vdsm-tool
+++ b/vdsm-tool/vdsm-tool
@@ -154,7 +154,6 @@
return tool_command[cmd]["command"](*args)
except vdsm.tool.UsageError as e:
print('Error: ', e, '\n', file=sys.stderr)
- print_command_usage(tool_command[cmd]["command"], sys.stderr)
return 1
except Exception:
traceback.print_exc(file=sys.stderr)
--
To view, visit http://gerrit.ovirt.org/28406
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I6df376664ffcda66ea19f66e28edc0521566c091
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
