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