-
Notifications
You must be signed in to change notification settings - Fork 37.7k
ci: Run ci configs on cirrus #19179
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
ci: Run ci configs on cirrus #19179
Conversation
Concept ACK. |
3e22eb3
to
e0d6b75
Compare
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
Concept ACK: anything that allows us to get back CI detection of uses of uninitialized memory (UUM) is worth doing. The current unfortunate situation with no Valgrind and no MemorySanitzer in our main CI feels a bit scary TBH :\ |
8c8f0d4
to
dd99488
Compare
146147b
to
5c5d038
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK 969fcfc, I have reviewed the code and it looks OK, I agree it can be merged.
969fcfc
to
e19b968
Compare
e19b968
to
fa95c27
Compare
Rebased. Should be ready for merge |
fa95c27
to
fa7a438
Compare
ACK fa7a438: patch looks correct |
1e63e57 ci: disable Werror on MacOS build (not a backport) (Andrew Poelstra) 1852f83 ci: Bump macOS image to big-sur-xcode-12.5 (MarcoFalke) 4f33ba7 ci: Fix macOS brew install command (Hennadii Stepanov) 0265085 ci: Bump macOS VM image to the latest version (Hennadii Stepanov) 0740cee ci: Adjust cirrus ci task names (Andrew Poelstra) d08e584 ci: Fix doc typos in .cirrus.yml (Andrew Poelstra) 953cbe2 ci: Run nowallet ci config on cirrus (MarcoFalke) 60a5632 build: set minimum supported macOS to 10.14 (fanquake) Pull request description: Backport of bitcoin/bitcoin#20419 then bitcoin/bitcoin#19179 (except 2nd commit "Run i686 centos ci config on cirrus" which requires some more supporting PRs) then bitcoin/bitcoin#20545 then bitcoin/bitcoin#21542 then bitcoin/bitcoin#21663 then bitcoin/bitcoin#22122 We are getting 403 errors in our MacOS job on Cirrus, presumably because bintray.com has shut down (see discussion on upstream PR 21663). Backport this PR and several supporting ones. These PRs will be merged as part of the 22.0 rebase. In the interest of keeping CI running let's backport them early. Top commit has no ACKs. Tree-SHA512: cdfe96789adedbfb617cdafb534175323a8fd46606e67adf7053f97cc32712dd3ece850906b522b6b0bed137f52c1372319f52f4a7b303aec479ea54d213271c
Travis CI Org is shutting down, so move the configs to cirrus ci