]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
vstart_runner: upgrade the check for commands to be run as another user 28561/head
authorRishabh Dave <ridave@redhat.com>
Thu, 20 Jun 2019 16:05:38 +0000 (21:35 +0530)
committerRishabh Dave <ridave@redhat.com>
Thu, 20 Jun 2019 16:05:38 +0000 (21:35 +0530)
Rectify the condition that checks if command to be issued as another
user using sudo is issued as a single argument after "-c".

Signed-off-by: Rishabh Dave <ridave@redhat.com>
qa/tasks/vstart_runner.py

index e3589c912e43f9443000933a76d31b0f7b6bef2e..99c63a7c7b18ce2a667898f14d26f65dd8b437b1 100644 (file)
@@ -264,8 +264,8 @@ class LocalRemote(object):
         # the desired effect.
         errmsg = 'The entire command to executed as other user should be a ' +\
                  'single argument.\nargs - %s' % (args)
-        if ('sudo' in args or 'python' in args or 'python2' in args or
-           'python3' in args) and '-c' in args:
+        if 'sudo' in args and '-u' in args and '-c' in args and \
+           args.count('-c') == 1:
             if args.index('-c') != len(args) - 2 and \
                args[args.index('-c') + 2].find('-') == -1:
                 raise RuntimeError(errmsg)