-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Start using Cirrus CI #24597
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
Start using Cirrus CI #24597
Conversation
5686fcf
to
1ca320e
Compare
Giving up for now. For FreeBSD, I get the following error when running pip or python
so something is wrong with the install. For MacOS and Linux ARM64, the following happens
Not sure if they are related, but it would be nicer if the version numbering worked properly to start with... |
I see stuff like:
and
These are Debian system paths. Perhaps it would be best to follow that advice and create a |
This is because their checkout does a shallow clone without tags. |
Yes, it seems so on any branch other than |
May be worth modifying it so that it does cancel even on |
I tried to see if one somehow can trigger the runs based on setting a specific label, but to no avail. An option is to use manual https://cirrus-ci.org/guide/writing-tasks/#manual-tasks and then do a bit of a guessing game if the PR may affect anything... (Not sure exactly who can trigger it though.) |
This is fixed now by adding the
I'm not sure we should try to optimize too much before finding out if we're going to end up close to the limit. |
Turns out if we wait long enough, GHA has M1 now... |
It looks like those are xlarge runners, which are only available for GitHub Team or GitHub Enterprise Cloud plans. So we'd still need to wait some more. |
570d3c5
to
c6d61ec
Compare
Comparing the previous build on aarch64, it took >2min to build, and now takes 34s, a factor of about 4, which matches CPU count. So the Meson build does seem to be taking advantage of all CPUs as expected. On macOS, it was 43s and now 15s, which is only 3x, but that might be too fast already. Oddly, there are new image failures on aarch64. I'm not sure why that's happening as that test image doesn't seem to have changed. Perhaps it's about optimizations or similar. |
It appears we have waited long enough, at least for M1 macOS: #27723 |
Co-authored-by: Elliott Sales de Andrade <quantum.analyst@gmail.com>
Is it worth closing this now we can get native M1 runners on GH actions? #27723 |
Unless we think that aarch64-Linux is worthwhile, I'd be happy to close this. |
aarch64-linux runners are available on GH actions now now: https://github.blog/changelog/2025-01-16-linux-arm64-hosted-runners-now-available-for-free-in-public-repositories-public-preview/, so I think this has become redundant? |
Yes, I think unfortunately or fortunately, this PR is no longer needed. |
PR Summary
I expect quite a few attempts...
PR Checklist
Documentation and Tests
pytest
passes)Release Notes
.. versionadded::
directive in the docstring and documented indoc/users/next_whats_new/
.. versionchanged::
directive in the docstring and documented indoc/api/next_api_changes/
next_whats_new/README.rst
ornext_api_changes/README.rst