Skip to content

ci: Add Ubuntu and Windows arm jobs #3651

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

MSP-Greg
Copy link
Member

Description

GHA now supports arm64 runners on Ubuntu & Windows.

Added jobs for both.

Note that the Ubuntu jobs are affected by an unsigned vs signed char bug, probably in ragel, as the failures related to the c source code generated by it. I tried a few things, and got Ruby 2.7 passing, but the other versions failed. Gave up.

Disabled the 'PUMA_MAKE_WARNINGS_INTO_ERRORS=true' workflow step in the Ubuntu arm64 jobs, and the jobs passed...

Windows arm had issues with TestPumaServertest_accept_emfile (retries & intermittent failures), added a Puma::IS_ARM constant, add used it to add a small sleep for the test.

Your checklist for this pull request

  • I have reviewed the guidelines for contributing to this repository.
  • I have added (or updated) appropriate tests if this PR fixes a bug or adds a feature.
  • My pull request is 100 lines added/removed or less so that it can be easily reviewed.
  • If this PR doesn't need tests (docs change), I added [ci skip] to the title of the PR.
  • If this closes any issues, I have added "Closes #issue" to the PR description or my commit messages.
  • I have updated the documentation accordingly.
  • All new and existing tests passed, including Rubocop.

@MSP-Greg
Copy link
Member Author

The turbo-rails failures are due to changes in Ruby head with the Set class.

@github-actions github-actions bot added the waiting-for-review Waiting on review from anyone label Apr 26, 2025
@dentarg
Copy link
Member

dentarg commented Apr 28, 2025

The turbo-rails failures are due to changes in Ruby head with the Set class.

Do you have a link? Or is it broken and will be fixed?

@MSP-Greg
Copy link
Member Author

Do you have a link? Or is it broken and will be fixed?

See below. Set was changed in master. CI may pass once the new Ruby master builds are created (happens daily).

ruby/ruby#13074

ruby/ruby#13185

@dentarg
Copy link
Member

dentarg commented Apr 29, 2025

I see, ruby/ruby#13074 had a bug but ruby/ruby#13185 will fix it.

@MSP-Greg
Copy link
Member Author

@dentarg

And the bug fix has finally landed in the GHA Ruby master builds. So, turbo-rails is now passing.

@MSP-Greg
Copy link
Member Author

MSP-Greg commented May 3, 2025

Rebased to force a full CI run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI / Testing waiting-for-review Waiting on review from anyone
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants