Skip to content

Commit

Permalink
Move "Bazel Build" and "Bazel Test" GitHub actions to large runners.
Browse files Browse the repository at this point in the history
These GitHub actions, especially "Bazel Test" has been the bottleneck of our builds for a while now. A large part of that is due to the default runner only having 2 cores and thus only being able to run at most 2 Bazel actions at a time. Things have also gotten worse due to the increasing number of tests in Dagger's codebase and the slowing down of tests due to the move to XProcessing Testing and possibly increased I/O due to golden files.

This CL moves our "Bazel Build" and "Bazel Test" GitHub actions onto the large test runners. The "Bazel Test" runner in particular now has 16 cores which should allow 8x more tests to run in parallel and we should see a similar improvement in the time it takes these GitHub actions to run.

RELNOTES=N/A
PiperOrigin-RevId: 578232997
  • Loading branch information
bcorso authored and Dagger Team committed Nov 3, 2023
1 parent 8801fd2 commit 426492d
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,14 +28,18 @@ jobs:
bazel-build:
name: 'Bazel build'
needs: validate-latest-dagger-version
runs-on: ubuntu-latest
runs-on:
group: large-runner
labels: linux
steps:
- uses: actions/checkout@v3
- uses: ./.github/actions/bazel-build
bazel-test:
name: 'Bazel tests'
needs: validate-latest-dagger-version
runs-on: ubuntu-latest
runs-on:
group: large-runner
labels: ubuntu-22.04-16core
steps:
- uses: actions/checkout@v3
- uses: ./.github/actions/bazel-test
Expand Down

0 comments on commit 426492d

Please sign in to comment.