-
Notifications
You must be signed in to change notification settings - Fork 942
errors after a restart after a successful splice #6533
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
Comments
@rustyrussell rc2 while this issue hasn't been fixed? Is that because splicing is still experimental or has it been fixed in some other commit? When fixed I would like to test splicing again. |
I think just because no one put this in the roadmap sorry |
Sorry, I did miss this! Tests for splicing are badly lacking:( |
Thanks for the report 🙏 Can you post the logs from when the unilateral close started, around timestamp 2023-08-08T06:22:55.383Z? Specifically looking to see if "Restarting onchaind for channel..." appears beforehand, meaning it was caused by a lightningd restart flow or if it happened some other way. |
Currently in train but here is I think what you asked for:
|
Home now. Here is everything:
|
PR #6564 should specifically address the crash on restart issues. Still need to diagnose missing nodeid issue. |
PR #6564 will prevent the creation of unneeded justice transactions but it wont clear out ones that are already there. You shouldn't make a habit of it but if you need to get into the node to recover your funds, you can remove the queued up justice transaction. You can do that by going into the database and delete the (hopefully single) row in table |
@ddustin Thanks a lot. I haven't lost any funds and opened a new channel between the two nodes and all is running as before I tried the splice. I will retry the splice after the fix has been merged and report back. |
Add a regression check for Issue ElementsProject#6533
Add a regression check for Issue ElementsProject#6533
Add a regression check for Issue ElementsProject#6533
Add a regression check for Issue ElementsProject#6533
Add a regression check for Issue ElementsProject#6533
Add a regression check for Issue ElementsProject#6533
Uh oh!
There was an error while loading. Please reload this page.
Issue and Steps to Reproduce
I managed to add 100 ksats to a channel between 2 of my nodes (and made a reddit post about it!). It all went smooth and the nodes and channel were working fine (was able to pay invoices).
Before splicing:
100 ksats (main node) <=> 100 ksats (test node)
after
200 ksats (main node) <=> 100 ksats (test node)
I then tried to restart my test node (because the nodeid was no longer present in
listnodes
at my main node side of the channel (why?)) and got this:EDIT 1: removed the gossip_store and restarted my test node. Now I see this:
getinfo
outputBoth: are version
23.08rc1-29-g54bcb10
withConfig (test node)
Config (main node)
EDIT 2: This is what
listpeerchannels
shows on my test node:and this on my main node:
The text was updated successfully, but these errors were encountered: