CI: Remove non-working FreeBSD test.
What the developer wrote
Authored by Rusty Russell
CI: Remove non-working FreeBSD test.
It takes over an hour to run, because there's no FreeBSD wheel for the cryptography Python
package, and it builds it, in Rust, from scratch.
And eventually, takes over 125 minutes and times out.
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
What changed, and why it matters
This commit simply removes a slow, timing-out FreeBSD continuous-integration test job from the project's GitHub Actions. It does not change any production code, wallet logic, network handling, or cryptography. There is no security issue in the change itself.
No security action required. Optionally review whether FreeBSD support should be documented as best-effort or re-enable CI if a prebuilt cryptography wheel becomes available.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The deleted .github/workflows/bsd.yml defined a FreeBSD VM-based CI job that built the cryptography Python package from source (no FreeBSD wheel available), causing the 120-minute timeout to be exceeded. The patch removes the entire workflow file. No source code, build scripts, dependencies, or cryptographic operations used by Core Lightning are modified.
Changed components
.github/workflows/bsd.ymlInspect captured patch +0 / −79
diff --git a/.github/workflows/bsd.yml b/.github/workflows/bsd.yml
deleted file mode 100644
index 773cd325..00000000
--- a/.github/workflows/bsd.yml
+++ /dev/null
@@ -1,79 +0,0 @@
-name: FreeBSD Test
-
-on:
- push:
- branches:
- - "master"
- pull_request:
- workflow_dispatch:
-
-jobs:
- testfreebsd:
- runs-on: ubuntu-22.04
- name: Build and test on FreeBSD
- timeout-minutes: 120
- strategy:
- fail-fast: true
- matrix:
- bitcoind-version: ["27.1"]
- steps:
- - uses: actions/checkout@v4
- - name: Test in FreeBSD
- id: test
- uses: vmactions/freebsd-vm@v1
- with:
- usesh: true
- sync: rsync
- copyback: false
- prepare: |
- pkg install -y \
- bash \
- wget \
- python310 \
- gmake \
- git \
- python \
- postgresql16-server \
- autoconf \
- automake \
- libtool \
- bash \
- gettext \
- sqlite3 \
- libsodium \
- lowdown \
- pkgconf \
- jq \
- protobuf \
- curl \
- uv \
- libffi
-
- python3.10 -m ensurepip
- python3.10 -m pip install --upgrade pip
- curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y --default-toolchain stable
-
-
- run: |
- PATH=/root/.local/bin:$PATH:/root/.cargo/bin; export PATH
- uv sync --all-extras --all-groups
-
- git clone https://github.com/lightning/bolts.git ../bolts
- # fatal: unsafe repository ('/Users/runner/work/lightning/lightning' is owned by someone else)
- git config --global --add safe.directory `pwd`
- for d in libwally-core gheap jsmn libbacktrace; do git config --global --add safe.directory `pwd`/external/$d; done
- git submodule update --init --recursive
-
- ./configure CC="$CC" --disable-valgrind
-
- cat config.vars
-
- cat << EOF > pytest.ini
- addopts=-p no:logging --color=yes --timeout=1800 --timeout-method=thread --test-group-random-seed=42 --junitxml=report.xml --json-report --json-report-file=report.json --json-report-indent=2
- markers =
- slow_test: marks tests as slow (deselect with '-m "not slow_test"')
- EOF
-
- # Just run a "quick" test without memory checking
- uv run gmake
-
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.