-
Notifications
You must be signed in to change notification settings - Fork 262
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
Lack of CI testing on Apple Silicon-based devices #8000
Comments
CCing people we have discussed this before with and others who may not have heard about this issue yet to raise awareness about the problem: @feliperodri @tautschnig @kroening @martin-cs @remi-delmas-3000 |
In Kani, we have so far chosen to only check our release builds on this platform. Maybe that's a more feasible middle ground? At present, @NlightNFotis and @esteffin are involved in producing the release with high probability, so this might be practical? |
This will exercise builds on ARM/Apple Silicon. Fixes: diffblue#8000
This will exercise builds on ARM/Apple Silicon. Fixes: diffblue#8000
This will exercise builds on ARM/Apple Silicon. Fixes: diffblue#8000
Hello,
I'm writing this message to raise awareness of a current situation with regard to our CI configuration and our build processes.
We're currently lacking CI checks for systems based on Apple Silicon (M1, M2, M3, etc). This has caused a number of PRs being merged recently that while good on other platforms (i.e. passing all our existing checks) were failing on Apple Silicon devices, causing our
develop
branch onthe platform to fail.
The consequences of that have been thus far variable, ranging from minor nuisance to build-breaking bugs that failed on distributor's CI pipelines such as Homebrew's.
The reason for the current situation, from a historical standpoint has been twofold:
Both of these situations have changed as of late:
and among our users/developers.
Unfortunately, Github has only released the runners as part of the so called "large" runners, which require additional payment (normal runners are free for open source projects like CBMC), which requires us to do a cost-analysis (as they are charged on a per-minute of runtime basis) before we can activate them, with any decision unable to be made before we get there.
As a temporary remedy, and in-lack of a CI array for the platform, we would strongly recommend that all PRs should have a note if they have been tested on an Apple Silicon-based device. If there is no test on Apple Silicon then the PR should not be merged. You can ping @NlightNFotis and @esteffin who can test on Apple silicon if you do not have the resources yourself.
The text was updated successfully, but these errors were encountered: