Skip to content

Conversation

@waskow-consensys
Copy link
Contributor

@waskow-consensys waskow-consensys commented Oct 27, 2025

This repo has been identified as a higher cost repo in our Org.

Move runners to self hosted runners to mitigate cost and increase performance.

Thanks!


Note

Move PR and push workflows to a self-hosted Ubuntu runner, update OS condition checks, and add 30m timeouts to tagged tests.

  • CI (GitHub Actions)
    • PR workflow (.github/workflows/pr.yml):
      • Switch test job runner to gha-runner-scale-set-ubuntu-24-amd64-xxl.
      • Add -timeout=30m to tagged go test commands.
    • Push workflow (.github/workflows/push.yml):
      • Replace matrix OS ubuntu-latest-128 with gha-runner-scale-set-ubuntu-24-amd64-xxl.
      • Update Ubuntu checks to startsWith(matrix.os, 'gha-runner-scale-set-ubuntu').
      • Gate solc deps install and Ubuntu-specific tests behind the new runner check.

Written by Cursor Bugbot for commit 9901895. This will update automatically on new commits. Configure here.

cursor[bot]

This comment was marked as outdated.

@ivokub
Copy link
Collaborator

ivokub commented Oct 28, 2025

This repo has been identified as a higher cost repo in our Org.

Move runners to self hosted runners to mitigate cost and increase performance.

Thanks!

Note

Switch test jobs to a self-hosted Ubuntu runner label across PR and push workflows.

  • CI (GitHub Actions):

    • Update test job runners to self-hosted gha-runner-scale-set-ubuntu-22.04-amd64-xxl.

      • /.github/workflows/pr.yml: runs-on changed for test job.
      • /.github/workflows/push.yml: matrix os replaces ubuntu-latest-128 with gha-runner-scale-set-ubuntu-22.04-amd64-xxl.

Written by Cursor Bugbot for commit 995a9c8. This will update automatically on new commits. Configure here.

Sure, I think we wouldn't have a issue changing runners. However what are the specs of the machine? Some tests are starting to timeout which didn't before: https://github.com/Consensys/gnark/actions/runs/18847896561/job/53845181796#step:7:47

We'll discuss maybe it is worth it to segregate the tests to have a fast path for PRs and then longer-running tests for end-to-end integration.

Copy link
Collaborator

@ivokub ivokub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Lets try it out on master as well.

@ivokub ivokub merged commit ddcae9d into master Nov 14, 2025
12 checks passed
@ivokub ivokub deleted the self_hosted_runners branch November 14, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants