Skip to content

Commit

Permalink
Attempt #2: switch high-perf-docker to runs-on.com runners (aptos-lab…
Browse files Browse the repository at this point in the history
  • Loading branch information
geekflyer authored Jun 28, 2024
1 parent ba06d09 commit ad42e03
Show file tree
Hide file tree
Showing 17 changed files with 28 additions and 28 deletions.
2 changes: 1 addition & 1 deletion .github/actionlint.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
self-hosted-runner:
# Labels of self-hosted runners in array of string
labels:
- high-perf-docker
- runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
2 changes: 1 addition & 1 deletion .github/workflows/cli-e2e-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
# we ensure that the Aptos CLI works with all 3 prod networks, at least
# based on the tests in the test suite.
run-cli-tests:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
permissions:
contents: read
id-token: write
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/coverage-move-only.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ concurrency:
jobs:
rust-move-unit-coverage:
timeout-minutes: 60
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/rust-setup@main
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/coverage.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
(github.event_name == 'schedule' && github.ref_name == 'main')
# Note the tests run slowly due to instrutmentation. It takes CI 10 hrs
timeout-minutes: 720
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
with:
Expand Down Expand Up @@ -58,7 +58,7 @@ jobs:
contains(github.event.pull_request.labels.*.name, 'CICD:run-coverage') ||
(github.event_name == 'schedule' && github.ref_name == 'main')
timeout-minutes: 720 # incremented from 240 due to execution time limit hit in cron
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
with:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docker-build-rosetta.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ permissions:

jobs:
build:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/faucet-tests-main.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
# be compatible in production.
run-tests-main:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
if: ${{ !inputs.SKIP_JOB }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/faucet-tests-prod.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ jobs:
run-tests-devnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/docker-setup@main
Expand All @@ -57,7 +57,7 @@ jobs:
run-tests-testnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
permissions:
contents: read
id-token: write
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ on:

jobs:
find-packages-with-undeclared-feature-dependencies:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/rust-setup@main
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/indexer-grpc-integration-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
run-tests-local-testnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
env:
# spin up the local testnet using the latest devnet image
VALIDATOR_IMAGE_REPO: ${{ vars.GCP_DOCKER_ARTIFACT_REPO }}/validator
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/keyless-circuit-daily-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ concurrency:

jobs:
run-all-circuit-tests:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
timeout-minutes: 30
steps:
- uses: actions/checkout@v3
Expand Down
16 changes: 8 additions & 8 deletions .github/workflows/lint-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ jobs:
# Run the crypto hasher domain separation checks
rust-cryptohasher-domain-separation-check:
needs: file_change_determinator
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
steps:
- uses: actions/checkout@v4
Expand All @@ -69,7 +69,7 @@ jobs:
# Run all rust lints. This is a PR required job.
rust-lints:
needs: file_change_determinator
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v4
if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
Expand All @@ -94,7 +94,7 @@ jobs:
github.event.pull_request.auto_merge != null) ||
contains(github.event.pull_request.body, '#e2e'
)
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v4
if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
Expand All @@ -109,7 +109,7 @@ jobs:
# Run only the targeted rust unit tests. This is a PR required job.
rust-targeted-unit-tests:
needs: file_change_determinator
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v4
with:
Expand All @@ -128,7 +128,7 @@ jobs:
github.event_name == 'push' ||
contains(github.event.pull_request.labels.*.name, 'CICD:run-all-unit-tests')
)
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v4
- name: Run rust unit tests
Expand All @@ -147,7 +147,7 @@ jobs:
contains(github.event.pull_request.labels.*.name, 'CICD:run-e2e-tests') ||
github.event.pull_request.auto_merge != null
)
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v4
if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
Expand All @@ -162,7 +162,7 @@ jobs:

# Run the consensus only unit tests
rust-consensus-only-unit-test:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
if: contains(github.event.pull_request.labels.*.name, 'CICD:build-consensus-only-image')
steps:
- uses: actions/checkout@v4
Expand All @@ -178,7 +178,7 @@ jobs:

# Run the consensus only smoke test
rust-consensus-only-smoke-test:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
if: contains(github.event.pull_request.labels.*.name, 'CICD:build-consensus-only-image')
steps:
- uses: actions/checkout@v4
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/move-test-compiler-v2.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ concurrency:
jobs:
# Run Aptos Move Compiler v2 tests. This is a PR required job.
rust-move-tests:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- name: Run Aptos Move tests with compiler V2
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/node-api-compatibility-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ jobs:
# if there are any changes that would affect it within the PR / commit. If
# everything is checked in, run tests, build the SDK, and upload it to npmjs.
node-api-compatibility-tests:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
permissions:
contents: read
id-token: write
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/prover-daily-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ concurrency:

jobs:
prover-inconsistency-test:
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
timeout-minutes: ${{ github.event_name == 'pull_request' && 10 || 480}}
steps:
- uses: actions/checkout@v3
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/run-gas-calibration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ concurrency:
jobs:
run-gas-calibration:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
with:
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/rust-client-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
run-tests-devnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/docker-setup@main
Expand All @@ -50,7 +50,7 @@ jobs:
run-tests-testnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/docker-setup@main
Expand All @@ -69,7 +69,7 @@ jobs:
run-tests-mainnet:
if: contains(github.event.pull_request.labels.*.name, 'CICD:non-required-tests')
needs: [permission-check]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
- uses: aptos-labs/aptos-core/.github/actions/docker-setup@main
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/ts-sdk-e2e-tests.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ jobs:
# Now that the latter runs against the local testnet too we make these land blocking.
run-tests-main-branch:
needs: [permission-check, file_change_determinator]
runs-on: high-perf-docker
runs-on: runs-on,cpu=64,family=c7,hdd=500,image=aptos-ubuntu-x64,run-id=${{ github.run_id }}
steps:
- uses: actions/checkout@v3
if: needs.file_change_determinator.outputs.only_docs_changed != 'true'
Expand Down

0 comments on commit ad42e03

Please sign in to comment.