Bump test dependencies and reorganize Docker files#869
Bump test dependencies and reorganize Docker files#869tnull merged 1 commit intolightningdevkit:mainfrom
Conversation
|
I've assigned @tnull as a reviewer! |
|
🔔 1st Reminder Hey @tnull! This PR has been waiting for your review. |
tnull
left a comment
There was a problem hiding this comment.
Thanks!
- bitcoind/corepc-node: 27.2 → 29.0 (corepc-node 0.10.0 → 0.10.1)
Hmm, remind me, why can't we bump to v30 or even v31 directly? Was it something to do with LND not being compatible yet? If so, is the issue already fixed on their main and could we intermittently opt to use a more recent (e.g. 'daily-testing') image to work around this?
|
What I found is that corepc-node 0.10.1 doesn't have a 30_0 feature flag yet, and since our tests use it to manage the regtest bitcoind instance, I went with 29.0. |
Test dependency updates: - bitcoind/corepc-node: 27.2 → 29.0 (corepc-node 0.10.0 → 0.10.1) - lnd_grpc_rust: 2.10.0 → 2.14.0 - Docker CLN: v23.08 → v25.12.1 (elementsproject/lightningd) - Docker LND: v0.18.5-beta → v0.20.1-beta - Docker bitcoind (CLN/LND compose): 27.2 → 29.1 - Download script: bitcoind 29.0 with updated SHA256 hashes CI fixes for bitcoind 29.0: - Cache key: add version to invalidate stale 27.2 cache - mkdir -p: needed when bitcoind cache misses but electrs cache hits CLN v25 compatibility: - Remove --experimental-anchors (deprecated in v25, causes BROKEN startup) - Add sleep before close_channel (monitor update timing with v25+) Organization: - Move docker-compose files to tests/docker/ (resolves lightningdevkit#855) Closes lightningdevkit#855
eef5142 to
1f5e920
Compare
Ah, I thought they had shipped v30 support yet. Makes sense then! |
|
Okay I will! |
Test dependency updates:
Organization:
tests/docker/(resolves Move interop test Docker files to tests/docker/ #855)Closes #855 and Prerequisite for #839 (interop-test-harness).