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
Add regtest support #249
Add regtest support #249
Conversation
b2c1a52
to
08adb5c
Compare
Remove unsupported option 'testnet'.
bind-addr can't be null.
Enables lightning-cli group access when nonstandard config options are set.
Also: - quote paths - use long form args
Also set bitcoind rpc port.
Needed for regtest scenario.
Needed for regtest scenario.
Rebased to master. |
ACK. This is excellent! Feeling a bit useless because I have nothing to complain about.
Yeah generally that seems better but can be different PR. |
Ah, |
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.
ACK 9951f10
Regtest allows testing services in a synced-blockchain scenario.
Run the test via
test/run-tests.sh -s regtest
.Besides enabling in-depth service testing, this is also a great consistency check for our service modules. The number of service bugfixes included in this PR is testament to this.
regtest
revealed that lightning-loop always requiresserver.host
to be set. Should we add an option for it?