Skip to content

Commit

Permalink
Use arm-medium runners but turn up test threads
Browse files Browse the repository at this point in the history
Using instances with 16 vCPUs is a bit silly for CI, but we can jack up the number of threads used for integration tests to match num of CPUs plus one
  • Loading branch information
dzuelke committed Jun 4, 2024
1 parent 4a65142 commit 11f683b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ jobs:
arch: "arm64"
- builder: "builder:20"
arch: "arm64"
runs-on: ${{ matrix.arch == 'arm64' && 'pub-hk-ubuntu-22.04-arm-large' || 'ubuntu-latest' }}
runs-on: ${{ matrix.arch == 'arm64' && 'pub-hk-ubuntu-22.04-arm-medium' || 'ubuntu-latest' }}
env:
INTEGRATION_TEST_CNB_BUILDER: heroku/${{ matrix.builder }}
steps:
Expand All @@ -75,4 +75,4 @@ jobs:
# only those and not the unit tests, via the `--ignored` option. On the latest stack
# we run all integration tests, but on older stacks we only run stack-specific tests.
- name: Run integration tests (all tests)
run: cargo test --locked -- --ignored --test-threads 5
run: cargo test --locked -- --ignored --test-threads $(($(nproc)+1))

0 comments on commit 11f683b

Please sign in to comment.