Skip to content

Commit

Permalink
Update perf-testing-setup.yml
Browse files Browse the repository at this point in the history
  • Loading branch information
YunchuWang authored Jan 5, 2024
1 parent fb4b719 commit 88f8422
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/perf-testing-setup.yml
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,8 @@ jobs:
run: |
echo "Starting py-spy in background..."
echo "Container id is $container_id"
docker exec -it $container_id sh -c "pip install py-spy"
spy_pid=$(docker exec -it $container_id sh -c "py-spy record -p $worker_pid -o helloperf.svg -f flamegraph --idle --nonblocking --rate 1500")
docker exec $container_id sh -c "pip install py-spy"
spy_pid=$(docker exec $container_id sh -c "py-spy record -p $worker_pid -o helloperf.svg -f flamegraph --idle --nonblocking --rate 1500")
echo "Py-spy process id is $spy_pid"
echo "spy_pid=$spy_pid" >> $GITHUB_ENV
Expand All @@ -92,7 +92,7 @@ jobs:
chmod 755 .ci/perf_tests/run-perftests.sh
.ci/perf_tests/run-perftests.sh localhost $PORT ${{ env.TESTS_DIR_PATH }} ${{ matrix.test_to_run }}
echo "Tests completed, terminating py-spy..."
docker exec -it $container_id sh -c "kill $spy_pid"
docker exec $container_id sh -c "kill $spy_pid"
docker cp $container_id:helloperf.svg .
- name: Upload profiling result to artifact
Expand Down

0 comments on commit 88f8422

Please sign in to comment.