Skip to content

ci: Temporarily move CI from Cirrus to GHA#35140

Closed
maflcko wants to merge 1 commit into
bitcoin:masterfrom
maflcko:2604-ci-ugh
Closed

ci: Temporarily move CI from Cirrus to GHA#35140
maflcko wants to merge 1 commit into
bitcoin:masterfrom
maflcko:2604-ci-ugh

Conversation

@maflcko

@maflcko maflcko commented Apr 22, 2026

Copy link
Copy Markdown
Member

Cirrus Runners was purchased by OpenAI, which promptly shut it down (dig cirrus-runners.app is empty).

So temporarily move the runners from OpenAI to Microsoft.

@DrahtBot DrahtBot added the Tests label Apr 22, 2026
@DrahtBot

DrahtBot commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/35140.

Reviews

See the guideline for information on the review process.

Type Reviewers
Stale ACK achow101, m3dwards

If your review is incorrectly listed, please copy-paste <!--meta-tag:bot-skip--> into the comment that the bot should ignore.

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #35397 (ci: add OpenBSD Clang cross job by fanquake)
  • #35378 (ci: switch to warp runners by willcl-ark)

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.

@maflcko

maflcko commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

LLM disclaimer: This diff was written by OpenAI Codex

@achow101

Copy link
Copy Markdown
Member

I thought they were supposed to keep going until June 1, but it looks like the runners are gone... All current CI tasks that used their runners are stuck on "waiting for a runner".

ACK fa43cfd

@maflcko

maflcko commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

Yeah, idk either. I can also only see that it is down, but we'll have to do the switch anyway, and this is just a temporary workaround.

If the CI passes here, but not otherwise, it seems fine to merge, but should also be fine to wait longer.

@maflcko

maflcko commented Apr 22, 2026

Copy link
Copy Markdown
Member Author

timed out ...

@maflcko
maflcko marked this pull request as draft April 22, 2026 19:07
@m3dwards

Copy link
Copy Markdown
Contributor

timed out ...

Interesting as it has been working on forks. I wonder if a primed cache would have helped. I see you have updated the timeouts now which seems reasonable.

@m3dwards

Copy link
Copy Markdown
Contributor

Concept ACK, even if this only a very temporary solution. Things will back up and be very slow but some runs is better than no runs.

@m3dwards

m3dwards commented Apr 22, 2026

Copy link
Copy Markdown
Contributor

ACK fae494a - With the understanding that Cirrus Runners are unavailable

@DrahtBot

Copy link
Copy Markdown
Contributor

Guix builds (on x86_64) [untrusted test-only build, possibly unsafe, not for production use]

File commit ed1795a
(master)
commit 00957e3
(pull/35140/merge)
*-aarch64-linux-gnu-debug.tar.gz ec6bf976fbc4a6dc... e3d37c3c0f133094...
*-aarch64-linux-gnu.tar.gz 9ef2478545489cd3... dd5cd28f6bc81e21...
*-arm-linux-gnueabihf-debug.tar.gz 217cd259e3108ca9... 70d5e336a2e2d00b...
*-arm-linux-gnueabihf.tar.gz b09d0e0ddced8c45... 769fe7824de94d67...
*-arm64-apple-darwin-codesigning.tar.gz c093451e401a032e... d9f218021a9f6e41...
*-arm64-apple-darwin-unsigned.tar.gz 7e75ea2b7d8588c1... 11511c87718c4bef...
*-arm64-apple-darwin-unsigned.zip 5a9c0991f5f2b15d... a3315f9b86a8f018...
*-powerpc64-linux-gnu-debug.tar.gz 9a696cf5f3fac68e... 9c72e4af69c6c8df...
*-powerpc64-linux-gnu.tar.gz e8c09fd3257f456a... a92c9593771fb6c8...
*-riscv64-linux-gnu-debug.tar.gz 023d2d726ad5ab73... ad0f0e55f78d7551...
*-riscv64-linux-gnu.tar.gz 775c8fb1825f126d... a347ee975448ad20...
*-win64-codesigning.tar.gz 11fe6fdbaf135b64... 02a26556998a2256...
*-win64-debug.zip 83d2c770af683ce4... e5acb23407b20b67...
*-win64-setup-unsigned.exe 15c6f0c14a14720b... 2fea200affa28d98...
*-win64-unsigned.zip fbb5fb942d9e578a... f1bb5d9b213e86d9...
*-x86_64-apple-darwin-codesigning.tar.gz cad2f406d3dd725b... 45c67e3f396e9389...
*-x86_64-apple-darwin-unsigned.tar.gz ce208589b3586c1a... e4226c3cd8903eb1...
*-x86_64-apple-darwin-unsigned.zip cfef272372f104c4... 782069a50dbe23c5...
*-x86_64-linux-gnu-debug.tar.gz 826e73fa1099011e... 766759b2404a85ab...
*-x86_64-linux-gnu.tar.gz 110457e53798c80e... d46f83197e6bb121...
*.tar.gz 02f2d9bcd97e39a8... 628ce917d863d1bb...
SHA256SUMS.part 668f1e11d798aca1... 5ee4a13a8c7a4914...
guix_build.log 1bd008a100952d10... d4ba1f44b7a4f0f0...
guix_build.log.diff a65c2fda9feba4ab...

@maflcko

maflcko commented May 22, 2026

Copy link
Copy Markdown
Member Author

rebased for gha cache, which should be picked up here

@maflcko

maflcko commented May 22, 2026

Copy link
Copy Markdown
Member Author

Looks like the cache is working again. https://github.com/bitcoin/bitcoin/actions/runs/26287233799/job/77380011110?pr=35140#step:10:185:

+ docker buildx build --file=/home/runner/work/bitcoin/bitcoin/ci/test_imagefile --build-arg=CI_IMAGE_NAME_TAG=mirror.gcr.io/debian:trixie --build-arg=FILE_ENV=./ci/test/00_setup_env_arm.sh --build-arg=BASE_ROOT_DIR=/home/runner/work/_temp --platform=linux/arm64 --label=bitcoin-ci-test --tag=ci_arm_linux --cache-from type=gha,scope=ci_arm_linux --load /home/runner/work/bitcoin/bitcoin
#0 building with "builder-a629b1c4-df8d-4392-be07-d44f07b76ca1" instance using docker-container driver

#1 [internal] load build definition from test_imagefile
#1 transferring dockerfile:
#1 transferring dockerfile: 946B 0.0s done
#1 DONE 0.1s

#2 [internal] load metadata for mirror.gcr.io/debian:trixie
#2 DONE 1.8s

#3 [internal] load .dockerignore
#3 transferring context: 2B done
#3 DONE 0.1s

#4 [internal] load build context
#4 DONE 0.0s

#5 [1/5] FROM mirror.gcr.io/debian:trixie@sha256:4ae67669760b807c19f23902a3fd7c121a6a70cf2ae709035674b23e712e4d62
#5 resolve mirror.gcr.io/debian:trixie@sha256:4ae67669760b807c19f23902a3fd7c121a6a70cf2ae709035674b23e712e4d62 0.0s done
#5 DONE 0.1s

#6 importing cache manifest from gha:11209079260555040155
#6 DONE 0.2s

#4 [internal] load build context
#4 transferring context: 47.02kB done
#4 DONE 0.1s

#7 [3/5] COPY ./ci/test/00_setup_env.sh ././ci/test/00_setup_env_arm.sh ./ci/test/01_base_install.sh ./ci/test/01_iwyu.patch /ci_container_base/ci/test/
#7 CACHED

#8 [4/5] RUN sh -c "bash -c 'true' || ( apk update && apk add --no-cache bash )"
#8 CACHED

#9 [2/5] COPY ./ci/retry/retry /usr/bin/retry
#9 CACHED

#10 [5/5] RUN ["bash", "-c", "cd /ci_container_base/ && set -o errexit && source ./ci/test/00_setup_env.sh && DANGER_RUN_CI_ON_HOST=1 ./ci/test/01_base_install.sh"]
#10 CACHED

and further below:

Cacheable calls:     872 / 872 (100.0%)
  Hits:              871 / 872 (99.89%)

@maflcko

maflcko commented May 22, 2026

Copy link
Copy Markdown
Member Author

Hmm, actually the other caches are not working for some reason. https://github.com/bitcoin/bitcoin/actions/runs/26287233799/job/77380011168?pr=35140#step:5:46:

Prepare all required actions
Getting action download info
Download action repository 'actions/cache@v5' (SHA:27d5ce7f107fe9357f9df03efb73ab90386fccae)
Run ./.github/actions/restore-caches
Run actions/cache/restore@v5
Cache not found for input keys: ccache-ci_macos_cross-26287233799, ccache-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: depends-sources-ci_macos_cross-6af50d1e88ded1eb8c28fd886f8a6fd44c5851c793a63de32d3dabb5e561356a, depends-sources-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: depends-built-ci_macos_cross-6af50d1e88ded1eb8c28fd886f8a6fd44c5851c793a63de32d3dabb5e561356a, depends-built-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: previous-releases-ci_macos_cross-4036882c078a4ebb9f71a708667a51a3073a8316d45d1e1dd410a990e78660d7, previous-releases-ci_macos_cross-
Run echo "depends-sources-cache-hit=" >> $GITHUB_ENV

@fanquake

Copy link
Copy Markdown
Member

Maybe close for now, and we revisit if #35378 doesn't land?

@maflcko

maflcko commented May 27, 2026

Copy link
Copy Markdown
Member Author

Yeah, seems fine to close if that is merged.

Though, out of curiosity, I wonder why the caches are missed here (#35140 (comment)), considering that they were moved to GHA in #35348, and considering that this pull does not change anything about the caching.

@m3dwards

Copy link
Copy Markdown
Contributor

Hmm, actually the other caches are not working for some reason. https://github.com/bitcoin/bitcoin/actions/runs/26287233799/job/77380011168?pr=35140#step:5:46:

Prepare all required actions
Getting action download info
Download action repository 'actions/cache@v5' (SHA:27d5ce7f107fe9357f9df03efb73ab90386fccae)
Run ./.github/actions/restore-caches
Run actions/cache/restore@v5
Cache not found for input keys: ccache-ci_macos_cross-26287233799, ccache-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: depends-sources-ci_macos_cross-6af50d1e88ded1eb8c28fd886f8a6fd44c5851c793a63de32d3dabb5e561356a, depends-sources-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: depends-built-ci_macos_cross-6af50d1e88ded1eb8c28fd886f8a6fd44c5851c793a63de32d3dabb5e561356a, depends-built-ci_macos_cross-
Run actions/cache/restore@v5
Cache not found for input keys: previous-releases-ci_macos_cross-4036882c078a4ebb9f71a708667a51a3073a8316d45d1e1dd410a990e78660d7, previous-releases-ci_macos_cross-
Run echo "depends-sources-cache-hit=" >> $GITHUB_ENV

I don't know why this didn't hit but I can see some later jobs hitting this cache and building quickly: https://github.com/bitcoin/bitcoin/actions/runs/26527924738/job/78136523172

@DrahtBot

Copy link
Copy Markdown
Contributor

🐙 This pull request conflicts with the target branch and needs rebase.

@fanquake

Copy link
Copy Markdown
Member

Closing for now, given warp has been merged.

@fanquake fanquake closed this May 28, 2026
@maflcko
maflcko deleted the 2604-ci-ugh branch May 28, 2026 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants