Skip to content

scripts+GitHub: use bitcoind v30.0 #10689

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.

Sign up for GitHub

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

Open
ziggie1984 wants to merge 3 commits into
base: master
Choose a base branch
from
+23 −3
Open

scripts+GitHub: use bitcoind v30.0 #10689

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
752ad16
scripts+GitHub: use bitcoind v30.0
ziggie1984 Mar 29, 2026
c6c69d3
testing: allow patch versions of bitcoin core
ziggie1984 Apr 16, 2026
13e002d
lntest: pin pre-v30 mempool policy defaults in itest bitcoind
ziggie1984 Apr 21, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,8 @@ defaults:
shell: bash

env:
BITCOIN_VERSION: "29"
# Accepts either a major image tag like "30" or a patch tag like "29.1".
BITCOIN_VERSION: "30"

# TRANCHES defines the number of tranches used in the itests.
TRANCHES: 16
Expand Down
8 changes: 8 additions & 0 deletions lntest/bitcoind_common.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -181,6 +181,14 @@ func newBackend(miner string, netParams *chaincfg.Params, extraArgs []string,
//
// TODO: Remove once btcd supports v2 P2P transport.
"-v2transport=0",
// Pin the pre-v30 mempool policy defaults (1 sat/vB)
// so the itest suite keeps exercising the fee math it
// was written against. v30 lowered minrelaytxfee and
// incrementalrelayfee to 100 sat/kvB, which breaks
// integer sat/vByte assertions and alters RBF bump
// thresholds across the sweeper/bumpfee tests.
"-minrelaytxfee=0.00001",
"-incrementalrelayfee=0.00001",
}
cmdArgs = append(cmdArgs, extraArgs...)
bitcoind := exec.Command("bitcoind", cmdArgs...)
Expand Down
15 changes: 13 additions & 2 deletions scripts/install_bitcoind.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,18 @@ BITCOIND_VERSION=$1

# Useful for testing RCs: e.g. TAG_SUFFIX=.0rc1, DIR_SUFFIX=.0rc1
TAG_SUFFIX=
DIR_SUFFIX=.0

# The docker image tag and the install directory version don't always use the
# same format. Major-only tags like `29` or `30` install into
# `/opt/bitcoin-29.0` and `/opt/bitcoin-30.0`, while patch tags like `29.1`
# install into `/opt/bitcoin-29.1`.
if [[ "$BITCOIND_VERSION" == *.* ]]; then
BITCOIND_DIR_VERSION="$BITCOIND_VERSION"
else
BITCOIND_DIR_VERSION="${BITCOIND_VERSION}.0"
fi

DIR_SUFFIX=

# Useful for testing against an image pushed to a different Docker repo.
REPO=lightninglabs/bitcoin-core
Expand All @@ -19,5 +30,5 @@ fi

docker pull ${REPO}:${BITCOIND_VERSION}${TAG_SUFFIX}
CONTAINER_ID=$(docker create ${REPO}:${BITCOIND_VERSION}${TAG_SUFFIX})
sudo docker cp $CONTAINER_ID:/opt/bitcoin-${BITCOIND_VERSION}${DIR_SUFFIX}/bin/bitcoind /usr/local/bin/bitcoind
sudo docker cp $CONTAINER_ID:/opt/bitcoin-${BITCOIND_DIR_VERSION}${DIR_SUFFIX}/bin/bitcoind /usr/local/bin/bitcoind
docker rm $CONTAINER_ID
Loading