Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Etc ci performance run locally #12946

Merged
merged 17 commits into from Sep 6, 2016
Merged
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file
Failed to load files.

Always

Just for now

Removed shell option from subprocess.

  • Loading branch information
asajeffrey committed Aug 19, 2016
commit eb18665f5b8248e127398436208c28384b62fbcd
@@ -28,8 +28,7 @@ def execute_test(url, command, timeout):
print(' '.join(command))
print("Timeout:{}".format(timeout))
try:
return subprocess.check_output(command, stderr=subprocess.STDOUT,
shell=True, timeout=timeout)
return subprocess.check_output(command, stderr=subprocess.STDOUT, timeout=timeout)
except subprocess.CalledProcessError as e:
print("Unexpected Fail:")
print(e)
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.