From 21a1b30a3e215a95e91b58e0c288e036820f6020 Mon Sep 17 00:00:00 2001 From: Rishabh Dave Date: Thu, 20 Jun 2019 21:35:38 +0530 Subject: [PATCH] vstart_runner: upgrade the check for commands to be run as another user 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 --- qa/tasks/vstart_runner.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/qa/tasks/vstart_runner.py b/qa/tasks/vstart_runner.py index e3589c912e43f..99c63a7c7b18c 100644 --- a/qa/tasks/vstart_runner.py +++ b/qa/tasks/vstart_runner.py @@ -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) -- 2.39.5