Skip to content

Commit

Permalink
test: upgraded runner
Browse files Browse the repository at this point in the history
  • Loading branch information
erikburt committed Jan 3, 2025
1 parent b7ced6e commit 30849d5
Showing 1 changed file with 10 additions and 4 deletions.
14 changes: 10 additions & 4 deletions .github/workflows/ci-core-partial.yml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,6 @@ jobs:
run-unit-tests:
name: Tests (${{ matrix.type.test-suite }})
needs: filter
runs-on: ubuntu22.04-32cores-128GB
permissions:
id-token: write
contents: write
Expand All @@ -55,8 +54,15 @@ jobs:
- test-suite: "core"
module-directory: "./"
build-flags: "-tags=integration"
os: ubuntu-latest-64cores-256GB

Check failure on line 57 in .github/workflows/ci-core-partial.yml

View workflow job for this annotation

GitHub Actions / Validate Workflow Changes

1. This Ubuntu runner is 16-32 more expensive than a base Ubuntu runner. Consider using a smaller Ubuntu runner. (runner-ubuntu / error)
build-concurrency: "64"
run-concurrency: "32"
- test-suite: "ccip-deployment"
module-directory: "./deployment"
os: ubuntu22.04-32cores-128GB

Check failure on line 62 in .github/workflows/ci-core-partial.yml

View workflow job for this annotation

GitHub Actions / Validate Workflow Changes

1. This Ubuntu runner is 8-16 more expensive than a base Ubuntu runner. Consider using a smaller Ubuntu runner. (runner-ubuntu / error)
build-concurrency: "32"
run-concurrency: "16"
runs-on: ${{ matrix.type.os }}
steps:
- name: Checkout the repo
uses: actions/checkout@v4.2.1
Expand Down Expand Up @@ -94,7 +100,7 @@ jobs:
timeout-minutes: 10
with:
pipeline-step: "build"
build-concurrency: "32"
build-concurrency: ${{ matrix.type.build-concurrency }}
collect-coverage: ${{ needs.filter.outputs.should-collect-coverage }}
test-suite: ${{ matrix.type.test-suite }}
module-directory: ${{ matrix.type.module-directory }}
Expand All @@ -108,8 +114,8 @@ jobs:
CL_DATABASE_URL: ${{ env.DB_URL }}
with:
pipeline-step: "run"
run-concurrency: "16"
run-all-tests: ${{ needs.filter.outputs.should-run-all-tests }}
run-concurrency: ${{ matrix.type.run-concurrency }}
run-all-tests: "true" # ${{ needs.filter.outputs.should-run-all-tests }}
collect-coverage: ${{ needs.filter.outputs.should-collect-coverage }}
test-suite: ${{ matrix.type.test-suite }}
module-directory: ${{ matrix.type.module-directory }}
Expand Down

0 comments on commit 30849d5

Please sign in to comment.