Dima Kuznetsov has uploaded a new change for review.

Change subject: vdsm-tool: Add args to remove_saslpasswd
......................................................................

vdsm-tool: Add args to remove_saslpasswd

In a recent patch added, all commands receive arguments, starting with
their name, this patch changes signature to avoid this:
Traceback (most recent call last):
   File "/usr/bin/vdsm-tool", line 209, in main
     return tool_command[cmd]["command"](*args)

Change-Id: Id20a6a1ce8c54d4e757171edb31b22e6ceceb287
Signed-off-by: Dima Kuznetsov <dkuzn...@redhat.com>
---
M lib/vdsm/tool/passwd.py
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/vdsm refs/changes/16/29116/1

diff --git a/lib/vdsm/tool/passwd.py b/lib/vdsm/tool/passwd.py
index 9a2adc8..d490197 100644
--- a/lib/vdsm/tool/passwd.py
+++ b/lib/vdsm/tool/passwd.py
@@ -54,10 +54,14 @@
 
 
 @expose("remove-saslpasswd")
-def remove_saslpasswd():
+def remove_saslpasswd(*args):
     """
     Remove vdsm password for libvirt connection
     """
+
+    if args[1:]:
+        raise ExtraArgsError()
+
     rc, out, err = utils.execCmd(
         (
             constants.EXT_SASLPASSWD2,


-- 
To view, visit http://gerrit.ovirt.org/29116
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id20a6a1ce8c54d4e757171edb31b22e6ceceb287
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuzn...@redhat.com>
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to