-
Notifications
You must be signed in to change notification settings - Fork 2.2k
chain: add testnet4 support #9620
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.
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
Conversation
Important Review skippedAuto reviews are limited to specific labels. 🏷️ Labels to auto review (1)
Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
builds fine, but cannot bootstrap:
don't know if related or not,
|
So it's an entirely new network. Someone needs to run DNS seed infrastructure for it, but none for the network currently exist at present.
I suspect some code is still using the testnet3 path when it should be using the testnet4 path (or you forgot to set the flag in |
@@ -22,6 +22,14 @@ var BitcoinTestNetParams = BitcoinNetParams{ | |||
CoinType: keychain.CoinTypeTestnet, | |||
} | |||
|
|||
// BitcoinTestNet4Params contains parameters specific to the 4th version of the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should also update lncli
to mention that testnet4
is an option for the --network
command.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep
$ lncli --network=testnet4 getinfo
[lncli] could not load global options: unknown network: testnet4
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did add that. Are you sure you re-compiled and installed lncli
from the branch?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, tried again, now lncli works, but lnd does not create admin.macaroon
~ $ lnd
2025-03-24 21:46:49.302 [WRN] LTND: Config 'dust-threshold' is deprecated, please remove it
2025-03-24 21:46:49.302 [INF] LTND: Version Info rev=d757bb version=0.18.99-beta commit=tor/v1.1.6-109-gd757bb51e debuglevel=production logging=info
2025-03-24 21:46:49.302 [INF] LTND: Network Info rev=d757bb active_chain=Bitcoin network=testnet4
2025-03-24 21:46:49.303 [INF] RPCS: RPC server listening on 0.0.0.0:10009
2025-03-24 21:46:49.335 [INF] RPCS: gRPC proxy started at 127.0.0.1:8080
2025-03-24 21:46:49.336 [INF] LTND: Opening the main database, this might take a few minutes...
2025-03-24 21:46:49.336 [INF] LTND: Opening bbolt database, sync_freelist=false, auto_compact=false
2025-03-24 21:46:49.351 [INF] LTND: Creating local graph and channel state DB instances
2025-03-24 21:46:49.360 [INF] CHDB: Checking for schema update: latest_version=33, db_version=33
2025-03-24 21:46:49.360 [INF] CHDB: Checking for optional update: prune_revocation_log=false, db_version=empty
2025-03-24 21:46:49.360 [INF] LTND: Database(s) now open (time_to_open=24.300106ms)!
2025-03-24 21:46:49.360 [INF] LTND: We're not running within systemd or the service type is not 'notify'
2025-03-24 21:46:49.360 [INF] LTND: Waiting for wallet encryption password. Use `lncli create` to create a wallet, `lncli unlock` to unlock an existing wallet, or `lncli changepassword` to change the password of an existing wallet and unlock it.
$ lncli --network=testnet4 getinfo
[lncli] could not load global options: unable to read macaroon path (check the network setting!): open /home/vlad/.lnd/data/chain/bitcoin/testnet4/admin.macaroon: no such file or directory
$ ls ~/.lnd/data/chain/bitcoin/testnet4
macaroons.db
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You need to unlock/create a wallet first.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Confirmed I can interact using lncli
on testent4 np.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🏮
tACK
We'll work on standing up DNS infrastructure in the background so nodes can boostrap.
Will a testnet4 external fee estimator URL, such as https://nodes.lightning.computer/fees/v1/btctestnet4-fee-estimates.json, also be provided ? |
if anyone wants to open a testnet4 channel, do 50/50, or want me to open to them, ping t.me/MiddleWayNode It will be up 24/7 at 03a17ec2c693c82d7b8d6616dcaf57cbfb0df506045612e08c9580b0c55dda2c1d@72elnf2w7ahgqkwhqfh3pjrxdtwdszpl3bqhly6ipx6agz4sc6kwkfid.onion:9735 |
Hi @guggero.
while in the log:
|
Can you create a new issue please? Might be a bug introduced by #9605. |
if anyone needs a channel peer on clearnet, feel free to connect and open a channel: 024c6e1edd12f0792d0c1ddda3abc6e2fde6bf89f2848e00cf8d6a58fabb6c3ab6@81.17.102.136:9735 |
We're already on |
Adds
testnet4
support tolnd
.Fixes #8966.
Depends on btcsuite/btcwallet#994 and lightninglabs/neutrino#311.