Skip to content

Commit 7bdbe60

Browse files
committed
lntest: pin pre-v30 mempool policy defaults in itest bitcoind
Bitcoind v30 lowered the default minrelaytxfee and incrementalrelayfee from 1000 sat/kvB (1 sat/vB) to 100 sat/kvB. The itest suite was written against the old defaults and the lower values cascade into: - integer sat/vByte assertions losing precision below 1 sat/vB, and - RBF bump thresholds that alter sweeper/bumpfee replacement timing. Pin the old defaults in the itest bitcoind backend so the existing tests keep passing without per-test adaptation. Running against the new defaults is still worth doing, but that is a separate exercise that should not be bundled with the v30 version bump.
1 parent 82af67b commit 7bdbe60

1 file changed

Lines changed: 8 additions & 0 deletions

File tree

lntest/bitcoind_common.go

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -181,6 +181,14 @@ func newBackend(miner string, netParams *chaincfg.Params, extraArgs []string,
181181
//
182182
// TODO: Remove once btcd supports v2 P2P transport.
183183
"-v2transport=0",
184+
// Pin the pre-v30 mempool policy defaults (1 sat/vB)
185+
// so the itest suite keeps exercising the fee math it
186+
// was written against. v30 lowered minrelaytxfee and
187+
// incrementalrelayfee to 100 sat/kvB, which breaks
188+
// integer sat/vByte assertions and alters RBF bump
189+
// thresholds across the sweeper/bumpfee tests.
190+
"-minrelaytxfee=0.00001",
191+
"-incrementalrelayfee=0.00001",
184192
}
185193
cmdArgs = append(cmdArgs, extraArgs...)
186194
bitcoind := exec.Command("bitcoind", cmdArgs...)

0 commit comments

Comments
 (0)