Dan Kenigsberg has submitted this change and it was merged.

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>
Reviewed-on: http://gerrit.ovirt.org/29116
Reviewed-by: Yaniv Bronhaim <ybron...@redhat.com>
Reviewed-by: Dan Kenigsberg <dan...@redhat.com>
Reviewed-by: mooli tayer <mta...@redhat.com>
---
M lib/vdsm/tool/passwd.py
1 file changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, but someone else must approve
  mooli tayer: Looks good to me, but someone else must approve
  Dima Kuznetsov: Verified
  Dan Kenigsberg: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id20a6a1ce8c54d4e757171edb31b22e6ceceb287
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dima Kuznetsov <dkuzn...@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: mooli tayer <mta...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to