Skip to content

Commit

Permalink
[benchmark] Use python3 for subprocesses, too
Browse files Browse the repository at this point in the history
After grpc#28963, the Python entry points run with "python3" found on the
PATH.

It makes sense to use the same for subprocesses, for two reasons:

- There might be no "python" in the PATH, as in my Mac with brew
- Having a different Python for subprocesses, especially if it's a
  Python 2, is asking for trouble.
  • Loading branch information
lepistone committed Nov 14, 2023
1 parent c09a008 commit 2349ce6
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tools/run_tests/run_performance_tests.py
Original file line number Diff line number Diff line change
Expand Up @@ -296,7 +296,7 @@ def build_on_remote_hosts(
# start port server locally
build_jobs.append(
jobset.JobSpec(
cmdline=["python", "tools/run_tests/start_port_server.py"],
cmdline=["python3", "tools/run_tests/start_port_server.py"],
shortname="local_start_port_server",
timeout_seconds=2 * 60,
)
Expand Down

0 comments on commit 2349ce6

Please sign in to comment.