openstack/rally-openstack
Code Issues Proposed changes

Merge "Avoid shell=True in subprocess"

Browse Source
This commit is contained in:
Jenkins 2017-03-28 13:04:54 +00:00 committed by Gerrit Code Review
commit b6a0ed6f24
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
rally/plugins/workload/siege.py
View File

@ -46,8 +46,9 @@ def generate_urls_list(instances):
def run():
instances = list(get_instances())
urls = generate_urls_list(instances)
out = subprocess.check_output("siege -q -t 60S -b -f %s" % urls,
shell=True, stderr=subprocess.STDOUT)
out = subprocess.check_output(
["siege", "-q", "-t", "60S", "-b", "-f", urls],
stderr=subprocess.STDOUT)
for line in out.splitlines():
m = SIEGE_RE.match(line)
if m: