Skip to content

Commit

Permalink
Merge pull request #897 from isucon/feature-show-logs-to-ci-workflow
Browse files Browse the repository at this point in the history
Add log display step to CI workflow after benchmarks execution
  • Loading branch information
catatsuy authored Dec 21, 2024
2 parents 582ed80 + 052e03a commit 7631f61
Showing 1 changed file with 5 additions and 5 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,11 @@ jobs:
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
- name: Show logs
run: |
cd webapp
docker compose logs
- name: Check benchmark result
run: |
if [ ! -f benchmark_output.json ]; then
Expand All @@ -104,11 +109,6 @@ jobs:
exit 1
fi
- name: Show logs
run: |
cd webapp
docker compose logs
- name: Fail if benchmark failed
if: env.BENCHMARK_FAILED == 'true'
run: |
Expand Down

0 comments on commit 7631f61

Please sign in to comment.