Skip to content

Run the FreeBSD tests on GitHub Actions instead of Cirrus - #1683

Merged
bjorn3 merged 1 commit into
rust-lang:mainfrom
neilpang:ci/freebsd-vmactions
Aug 5, 2026
Merged

Run the FreeBSD tests on GitHub Actions instead of Cirrus#1683
bjorn3 merged 1 commit into
rust-lang:mainfrom
neilpang:ci/freebsd-vmactions

Conversation

@neilpang

@neilpang neilpang commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Closes #1640

Cirrus CI shut down on 2026-06-01, which removed the FreeBSD coverage. This
runs the same commands in a FreeBSD VM booted under QEMU/KVM on a regular
ubuntu-latest runner.

Both workarounds the Cirrus task carried are kept verbatim:

Verification

https://github.com/neilpang/rustc_codegen_cranelift/actions/runs/30730256715

22815 tests passed across 43 test binaries, 0 failed
job time 15m32s

cache-after-prepare skips the rustup install on subsequent runs.

@rustbot

This comment has been minimized.

@bjorn3 bjorn3 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please remove the old cirrus ci config too. And please amend the commit message to remove the issue links.

View changes since this review

Comment thread .github/workflows/freebsd.yml Outdated
Comment thread .github/workflows/freebsd.yml Outdated
@clubby789

clubby789 commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Raised a fix to hopefully avoid needing the TLS workaround. rust-lang/rust#160522. Though I do want to note in my local VM testing I didn't encounter this error in FreeBSD 15, while I did in 13 and 14.

@neilpang
neilpang force-pushed the ci/freebsd-vmactions branch from bef180b to 392bf49 Compare August 5, 2026 02:08
@neilpang

neilpang commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

All three done.

  • .cirrus.yml removed.
  • The push filter is gone, and timeout-minutes is 60.
  • The commit message no longer carries the issue references; they are in
    the PR description instead.

On the timeout: this run took 15m25s, and the two before it 13m32s and
15m32s, so 60 leaves about four times the observed duration.

https://github.com/rust-lang/rustc_codegen_cranelift/actions/runs/30968473458/job/92187475340

@clubby789 I left LD_STATIC_TLS_EXTRA alone since the Readme documents it
as a FreeBSD requirement, but this job runs the 15.1 image, so if it would
help to know whether the workaround is still needed there I can drop it
and post the result.

Comment thread .github/workflows/freebsd.yml Outdated
Cirrus CI shut down on 2026-06-01, which removed the FreeBSD coverage.
Run the same commands in a FreeBSD VM booted under QEMU/KVM on a regular
ubuntu-latest runner, keeping both workarounds the Cirrus task carried.

Also removes .cirrus.yml, which no longer runs anywhere.
@neilpang
neilpang force-pushed the ci/freebsd-vmactions branch from 392bf49 to eb75f0c Compare August 5, 2026 10:19
@neilpang

neilpang commented Aug 5, 2026

Copy link
Copy Markdown
Contributor Author

Removed. rustc.yml and cranelift-release-branch.yml do not set it either,
so this matches half the repo already.

@bjorn3
bjorn3 merged commit 07f0561 into rust-lang:main Aug 5, 2026
25 checks passed
@bjorn3

bjorn3 commented Aug 5, 2026

Copy link
Copy Markdown
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CirrusCI will shut down on 2026-06-01

4 participants