Skip to content

Commit

Permalink
chore(ci): optimize disk usage in arm run (#8564)
Browse files Browse the repository at this point in the history
This was creating unnecessary ARM disks per user
  • Loading branch information
ludamad authored Sep 16, 2024
1 parent 6fae8f0 commit 33e6aa4
Showing 1 changed file with 8 additions and 5 deletions.
13 changes: 8 additions & 5 deletions .github/workflows/ci-arm.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
setup:
uses: ./.github/workflows/setup-runner.yml
with:
username: ${{ github.event.pull_request.user.login || github.actor }}
username: master
runner_type: builder-arm
secrets: inherit

Expand All @@ -49,24 +49,27 @@ jobs:
build-images:
needs: [setup, changes]
runs-on: ${{ github.event.pull_request.user.login || github.actor }}-arm
# Note we don't put the 'if' here as that would also apply to dependent jobs, instead we just copy it into each step
runs-on: master-arm
steps:
# permission kludge before checkout, see https://github.com/actions/checkout/issues/211#issuecomment-611986243
- run: sudo chown -R $USER:$USER /home/ubuntu/
if: needs.changes.outputs.build-images == 'true'
- uses: actions/checkout@v4
with: { ref: "${{ env.GIT_COMMIT }}" }
- uses: ./.github/ci-setup-action
if: needs.changes.outputs.build-images == 'true'
with:
concurrency_key: build-images-arm
- name: "Push Build Images If Changed"
if: ${{ needs.changes.outputs.build-images }}
if: needs.changes.outputs.build-images == 'true'
timeout-minutes: 40
run: |
earthly-ci --push ./build-images/+build
build:
needs: [build-images]
runs-on: ${{ github.event.pull_request.user.login || github.actor }}-arm
runs-on: master-arm
steps:
# permission kludge before checkout, see https://github.com/actions/checkout/issues/211#issuecomment-611986243
- run: sudo chown -R $USER:$USER /home/ubuntu/
Expand All @@ -84,7 +87,7 @@ jobs:
# all the non-bench end-to-end integration tests for aztec
e2e:
needs: [build]
runs-on: ${{ github.event.pull_request.user.login || github.actor }}-arm
runs-on: master-arm
steps:
# permission kludge before checkout, see https://github.com/actions/checkout/issues/211#issuecomment-611986243
- run: sudo chown -R $USER:$USER /home/ubuntu/
Expand Down

0 comments on commit 33e6aa4

Please sign in to comment.