Skip to content

Commit

Permalink
Switch all workflows to run on besu-arm64 machine (#37)
Browse files Browse the repository at this point in the history
* Switch all workflows to run on besu-arm64 machine

* Revert machine changes for image build workflows

* Revert machine changes for prover-testing.yml

* Revert machine changes for prover-testing.yml

---------

Signed-off-by: Bradley Bown <[email protected]>
bradbown authored Sep 19, 2024

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
1 parent 43612d5 commit ce440d2
Showing 16 changed files with 47 additions and 47 deletions.
10 changes: 5 additions & 5 deletions .github/workflows/all-tools.yml
Original file line number Diff line number Diff line change
@@ -17,7 +17,7 @@ env:

jobs:
changes:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Filter commit changes
outputs:
all-tools: ${{ steps.filter.outputs.all-tools }}
@@ -38,15 +38,15 @@ jobs:
store_image_name_and_tags:
uses: ./.github/workflows/reuse-store-image-name-and-tags.yml

check_image_tags_exist:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Check image tags exist
needs: [ changes, store_image_name_and_tags ]
if: ${{ needs.changes.outputs.all-tools == 'false' }}
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v4
- name: Check image tags exist
uses: ./.github/actions/check-image-tags-exist
with:
@@ -57,7 +57,7 @@ jobs:
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

all-tools-tag-only:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: All tools tag only
needs: [ changes, store_image_name_and_tags, check_image_tags_exist ]
if: ${{ github.event_name != 'pull_request' && needs.changes.outputs.all-tools == 'false' }}
6 changes: 3 additions & 3 deletions .github/workflows/bridge-ui-e2e-tests.yml
Original file line number Diff line number Diff line change
@@ -14,7 +14,7 @@ on:

jobs:
run-e2e-tests:
runs-on: ubuntu-22.04
runs-on: besu-arm64
steps:
- name: Checkout
uses: actions/checkout@v4
@@ -27,7 +27,7 @@ jobs:
with:
node-version: 18.17.0
pnpm-install-options: '--frozen-lockfile --prefer-offline'

- name: Install Playwright
run: pnpm dlx [email protected] install --with-deps

@@ -57,7 +57,7 @@ jobs:
E2E_TEST_SEED_PHRASE: "test test test test test test test test test test test junk"
E2E_TEST_WALLET_PASSWORD: "TestPassword!"
NEXT_PUBLIC_INFURA_ID: ${{ secrets.PUBLIC_BRIDGE_UI_INFURA_ID }}

- name: Archive Playwright report
uses: actions/upload-artifact@v3
if: failure()
2 changes: 1 addition & 1 deletion .github/workflows/bridge-ui-publish.yml
Original file line number Diff line number Diff line change
@@ -14,7 +14,7 @@ on:

jobs:
publish:
runs-on: ubuntu-latest
runs-on: besu-arm64
if: github.event_name != 'pull_request' || (github.event_name == 'pull_request' && github.event.pull_request.draft == false)
steps:
- name: Checkout
2 changes: 1 addition & 1 deletion .github/workflows/cache-docker-images.yml
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@ on:

jobs:
changes:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Filter commit changes
outputs:
cache-images: ${{ steps.filter.outputs.cache-images }}
2 changes: 1 addition & 1 deletion .github/workflows/codeql.yml
Original file line number Diff line number Diff line change
@@ -18,7 +18,7 @@ on:
jobs:
analyze:
name: Analyze
runs-on: ubuntu-latest
runs-on: besu-arm64
permissions:
actions: read
contents: read
14 changes: 7 additions & 7 deletions .github/workflows/load-test.yml
Original file line number Diff line number Diff line change
@@ -21,34 +21,34 @@ on:
required: false
type: string

concurrency:
concurrency:
group: load-test-${{ github.event.inputs.network }}-${{ github.ref }}
cancel-in-progress: false

jobs:
run-load-test:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Run Load Test
steps:
- name: Checkout
uses: actions/checkout@v3
with:
fetch-depth: 0
ref: ${{ github.head_ref }}

- uses: actions/setup-java@v4
with:
distribution: temurin
java-version: 17

- name: Setup Gradle
uses: gradle/actions/setup-gradle@v3

- name: Hide sensitive inputs
uses: levibostian/[email protected]
with:
exclude_inputs: network, file

- name: Determine Private Key
id: set_private_key
run: |
@@ -62,8 +62,8 @@ jobs:
echo "Using sepolia private key from secrets."
echo "PRIVATE_KEY=${{ secrets.SEPOLIA_LOAD_TEST_PRIVATE_KEY }}" >> $GITHUB_ENV
fi
- name: Load Test
run: |
echo "Network to execute load test on: ${{ github.event.inputs.network }}"
./gradlew :testing-tools:app:run --args="-request ${{ github.event.inputs.network }}/${{ github.event.inputs.file }} -pk $PRIVATE_KEY"
./gradlew :testing-tools:app:run --args="-request ${{ github.event.inputs.network }}/${{ github.event.inputs.file }} -pk $PRIVATE_KEY"
4 changes: 2 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
@@ -8,7 +8,7 @@ jobs:
uses: ./.github/workflows/reuse-store-image-name-and-tags.yml

filter-commit-changes:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Filter commit changes
outputs:
coordinator: ${{ steps.filter.outputs.coordinator }}
@@ -114,7 +114,7 @@ jobs:
prover_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_prover }}
traces_api_facade_image_tagged: ${{ needs.check-and-tag-images.outputs.image_tagged_traces_api_facade }}
secrets: inherit

# Comment out the auto build and release step below as the plugin release should be
# by manual Github action for versioning control
# finalized-tag-updater-jar-build-release:
Original file line number Diff line number Diff line change
@@ -21,7 +21,7 @@ on:
jobs:

build-linux:
runs-on: ubuntu-latest
runs-on: besu-arm64
steps:
- name: Checkout code
uses: actions/checkout@v4
@@ -86,7 +86,7 @@ jobs:
release_artefacts:
name: Release artefacts
needs: [ build-linux, build-mac-os]
runs-on: ubuntu-latest
runs-on: besu-arm64
steps:
- name: Load cached binaries
uses: actions/download-artifact@v4
4 changes: 2 additions & 2 deletions .github/workflows/prover-testing.yml
Original file line number Diff line number Diff line change
@@ -96,7 +96,7 @@ jobs:
needs:
- staticcheck
- test
runs-on: ubuntu-latest
runs-on: besu-arm64
steps:
- name: Notify slack -- workflow failed
id: slack
@@ -119,7 +119,7 @@ jobs:
needs:
- staticcheck
- test
runs-on: ubuntu-latest
runs-on: besu-arm64
steps:
- name: Notify slack -- workflow succeeded
id: slack
20 changes: 10 additions & 10 deletions .github/workflows/reuse-check-images-tags-and-push.yml
Original file line number Diff line number Diff line change
@@ -46,7 +46,7 @@ on:

jobs:
check_image_tags_exist:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Check image tags exist
outputs:
last_commit_tag_exists_coordinator: ${{ steps.check_image_tags_exist_coordinator.outputs.last_commit_tag_exists }}
@@ -59,8 +59,8 @@ jobs:
common_ancestor_commit_tag_exists_traces_api_facade: ${{ steps.check_image_tags_exist_traces_api_facade.outputs.common_ancestor_commit_tag_exists }}
steps:
- name: Checkout
uses: actions/checkout@v4
uses: actions/checkout@v4

- name: Check image tags exist for coordinator
uses: ./.github/actions/check-image-tags-exist
if: ${{ inputs.coordinator_changed == 'false' }}
@@ -71,7 +71,7 @@ jobs:
image_name: consensys/linea-coordinator
docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Check image tags exist for postman
uses: ./.github/actions/check-image-tags-exist
if: ${{ inputs.postman_changed == 'false' }}
@@ -82,7 +82,7 @@ jobs:
image_name: consensys/linea-postman
docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Check image tags exist for prover
uses: ./.github/actions/check-image-tags-exist
if: ${{ inputs.prover_changed == 'false' }}
@@ -93,7 +93,7 @@ jobs:
image_name: consensys/linea-prover
docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Check image tags exist for traces-api-facade
uses: ./.github/actions/check-image-tags-exist
if: ${{ inputs.traces_api_facade_changed == 'false' }}
@@ -106,7 +106,7 @@ jobs:
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

image_tag_push:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Tag and push images
needs: [ check_image_tags_exist ]
outputs:
@@ -117,7 +117,7 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4

- name: Tag and push coordinator image
id: image_tag_push_coordinator
uses: ./.github/actions/image-tag-and-push
@@ -133,7 +133,7 @@ jobs:
common_ancestor_commit_tag_exists: ${{ needs.check_image_tags_exist.outputs.common_ancestor_commit_tag_exists_coordinator }}
docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Tag and push postman image
id: image_tag_push_postman
uses: ./.github/actions/image-tag-and-push
@@ -149,7 +149,7 @@ jobs:
common_ancestor_commit_tag_exists: ${{ needs.check_image_tags_exist.outputs.common_ancestor_commit_tag_exists_postman }}
docker_username: ${{ secrets.DOCKERHUB_USERNAME }}
docker_password: ${{ secrets.DOCKERHUB_TOKEN }}

- name: Tag and push prover image
id: image_tag_push_prover
uses: ./.github/actions/image-tag-and-push
4 changes: 2 additions & 2 deletions .github/workflows/reuse-store-image-name-and-tags.yml
Original file line number Diff line number Diff line change
@@ -15,7 +15,7 @@ on:

jobs:
store_image_name_and_tags:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Compute version tags
env:
# REF_NAME: ${{ github.ref_name }}
@@ -50,4 +50,4 @@ jobs:
echo "LAST_COMMIT_TAG: ${{ steps.step2.outputs.LAST_COMMIT_TAG }}"
echo "DEVELOP_TAG: ${{ steps.step2.outputs.DEVELOP_TAG }}"
echo "COMMON_ANCESTOR_TAG: ${{ steps.step2.outputs.COMMON_ANCESTOR_TAG }}"
echo "UNTESTED_TAG_SUFFIX: ${{ steps.step2.outputs.UNTESTED_TAG_SUFFIX }}"
echo "UNTESTED_TAG_SUFFIX: ${{ steps.step2.outputs.UNTESTED_TAG_SUFFIX }}"
2 changes: 1 addition & 1 deletion .github/workflows/reuse-tag-without-untested-suffix.yml
Original file line number Diff line number Diff line change
@@ -22,7 +22,7 @@ on:

jobs:
tag-without-untested-suffix:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: tag without untested suffix
strategy:
matrix:
10 changes: 5 additions & 5 deletions .github/workflows/run-smc-tests.yml
Original file line number Diff line number Diff line change
@@ -9,17 +9,17 @@ on:

jobs:
run-contract-tests:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Run smart contracts tests
steps:
- uses: actions/checkout@v3
with:
fetch-depth: 0
ref: ${{ github.head_ref }}

- name: Setup nodejs environment
uses: ./.github/actions/setup-nodejs

- name: Check JS formatting
run: pnpm -F contracts run lint:ts

@@ -30,7 +30,7 @@ jobs:
run: pnpm -F contracts run coverage

solidity-format-check:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Solidity format check
steps:
- uses: actions/checkout@v3
@@ -42,4 +42,4 @@ jobs:
uses: ./.github/actions/setup-nodejs

- name: Check Solidity formatting
run: pnpm -F contracts run lint:sol
run: pnpm -F contracts run lint:sol
4 changes: 2 additions & 2 deletions .github/workflows/stale.yml
Original file line number Diff line number Diff line change
@@ -11,14 +11,14 @@ permissions:

jobs:
stale:
runs-on: ubuntu-latest
runs-on: besu-arm64
steps:
- uses: actions/stale@v8
with:
# Issues
days-before-issue-stale: -1 # Deactivate stale issues
days-before-issue-close: -1 # Deactivate stale issues

# PR
days-before-pr-stale: 30
days-before-pr-close: 7
2 changes: 1 addition & 1 deletion .github/workflows/traces-api-facade-testing.yml
Original file line number Diff line number Diff line change
@@ -17,7 +17,7 @@ on:

jobs:
run-tests:
runs-on: ubuntu-latest
runs-on: besu-arm64
name: Traces api facade tests
steps:
- name: Checkout
4 changes: 2 additions & 2 deletions .github/workflows/valid-audit-pr-has-tags.yml
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@ on:

jobs:
check:
runs-on: ubuntu-latest
runs-on: besu-arm64

steps:
- name: Checkout code
@@ -23,4 +23,4 @@ jobs:
if [[ ! $TAG_NAME =~ ^contract-audit-.*-.*$ ]]; then
echo "Error: The latest commit must be tagged with a tag that matches the format 'contract-audit-${firm}-${date}'"
exit 1
fi
fi

0 comments on commit ce440d2

Please sign in to comment.