Skip to content

Commit

Permalink
Merge pull request #900 from isucon/feature-refactor-ci-yml-docker-co…
Browse files Browse the repository at this point in the history
…mmand

efactor benchmark run command for better readability in CI workflow
  • Loading branch information
catatsuy authored Dec 29, 2024
2 parents 1584570 + 6108724 commit 40fcf1d
Showing 1 changed file with 14 additions and 1 deletion.
15 changes: 14 additions & 1 deletion .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,20 @@ jobs:
- name: Run the benchmark
continue-on-error: true
run: |
docker container run --add-host host.docker.internal:host-gateway -p 5678:5678 -p 7890:7890 -i isucari-benchmarker /bin/benchmarker -target-url http://host.docker.internal -data-dir /initial-data -static-dir /static -payment-url http://host.docker.internal:5678 -payment-port 5678 -shipment-url http://host.docker.internal:7890 -shipment-port 7890 | tee benchmark_output.json || echo "BENCHMARK_FAILED=true" >> $GITHUB_ENV
docker container run \
--add-host host.docker.internal:host-gateway \
-p 5678:5678 \
-p 7890:7890 \
-i isucari-benchmarker \
/bin/benchmarker \
-target-url http://host.docker.internal \
-data-dir /initial-data \
-static-dir /static \
-payment-url http://host.docker.internal:5678 \
-payment-port 5678 \
-shipment-url http://host.docker.internal:7890 \
-shipment-port 7890 \
| tee benchmark_output.json || echo "BENCHMARK_FAILED=true" >> $GITHUB_ENV
- name: Show logs
run: |
Expand Down

0 comments on commit 40fcf1d

Please sign in to comment.