Skip to content

Conversation

galargh
Copy link
Contributor

@galargh galargh commented Jun 12, 2025

Partially resolves #1775

This introduces the following changes:

  • changes the CI workflow to be triggered only on pushes to master and all pull requests
  • migrates the test release job back to a self-hosted runner (no GPU needed)
  • migrates the test no default features job back to a self-hosted runner (no GPU needed)
  • reintroduces the test gpu jobs and puts them on self-hosted runners

The test gpu jobs are currently put on g6e.xlarge machines because we only have limited number of resources (vCPUs) available. When we are granted the quota increase, I'll migrate the jobs to more powerful runners.

If approved, similar PRs will be created for other repositories we identified as needing self-hosted runners.

I tested the updated job here: https://github.com/filecoin-project/rust-fil-proofs/actions/runs/15614277342/job/43982899999

It will also be tested as part of the PR flow.

@galargh galargh requested review from vmx and BigLep June 12, 2025 16:39
@BigLep
Copy link
Member

BigLep commented Jun 12, 2025

  • migrates the test release job back to a self-hosted runner (no GPU needed)
  • migrates the test no default features job back to a self-hosted runner (no GPU needed)

I assume you mean github-hosted runners (since no special hardware needed)? Or I guess you saying you're moving back to a non-gpu self hosted runner.

BigLep
BigLep previously approved these changes Jun 12, 2025
Copy link
Member

@BigLep BigLep left a comment

Choose a reason for hiding this comment

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

Sweet. I'm not sure how you figured out the nvdia drier install stuff, but I'm glad you got it working. Thakns.

@galargh
Copy link
Contributor Author

galargh commented Jun 12, 2025

  • migrates the test release job back to a self-hosted runner (no GPU needed)
  • migrates the test no default features job back to a self-hosted runner (no GPU needed)

I assume you mean github-hosted runners (since no special hardware needed)? Or I guess you saying you're moving back to a non-gpu self hosted runner.

Yes, sorry, I should have been more clear. I meant moving to our non-gpu self-hosted runner.

Copy link
Contributor

@vmx vmx left a comment

Choose a reason for hiding this comment

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

See inline comment, the rest looks good.

@vmx vmx merged commit 1b36e72 into master Jun 15, 2025
10 checks passed
@vmx vmx deleted the self-hosted branch June 15, 2025 19:07
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.

Update "proofs" repos' CI to use self-hosted runners that will function in 2025H2+
3 participants