Skip to content
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

Channel reserve message? #597

Closed
kingonly opened this issue Jul 11, 2023 · 7 comments · Fixed by #603
Closed

Channel reserve message? #597

kingonly opened this issue Jul 11, 2023 · 7 comments · Fixed by #603
Assignees
Milestone

Comments

@kingonly
Copy link
Member

Screenshot_2023-07-11-16-38-31-00_9a3a9060ca72a38f8a1fec1d0c1a640e

I had is for a while, but now it's gone

@erdemyerebasmaz
Copy link
Collaborator

Yes @kingonly, that is the channel reserve message. It exists on breezmobile as well.

@kingonly
Copy link
Member Author

@erdemyerebasmaz

  1. We set the channel reserve to zero.
  2. As I wrote, the message disappeared after while. There's something strange going on.

@JssDWt
Copy link
Contributor

JssDWt commented Jul 14, 2023

I think this has to do with funds that are reserved for fees or htlcs. That will reduce maxPayableMsat.We calculate the 'reserve' by substracting the channel balance with maxPayableMsat. But that may not always be correct I think, or it might update too slow. The account state sometimes doesn't update very quickly after a payment it seems.

@roeierez
Copy link
Member

@JssDWt you are correct that this is related to how much greenlight "thinks" we can spend out of the channel. It is not the channel reserve but is related to the on-chain fees.
Do you see any misbehaviour of the SDK here? I think once anchor are there this will be resolved right?

@JssDWt
Copy link
Contributor

JssDWt commented Jul 14, 2023

@roeierez I think it's mostly the account state that updates too slowly.
July 9th @Nackoo2000 reported a big delay in the balance screen update.
June 29th @Nackoo2000 reported a similar message as the one in this issue. That one was due to a htlc being in-flight. Or a htlc that had been in flight shortly before. He had a 'channel reserve' with the size of approximately the payment amount. So I think that was the htlc itself that was counted as reserve.

@roeierez
Copy link
Member

Makes sense. So the htlc is still in-flight in greenlight despite payment succeeded?
We do sync immediately after payment success but the fact that @Nackoo2000 reported "payment not found" makes me think it is pending because although we sync all payments (including pending) our lookup looks for completed payment only (https://github.com/breez/breez-sdk/blob/main/libs/sdk-core/src/breez_services.rs#L603).
Probably that needs to be changed at first step (include pending payments in the lookup), what do you think?
cc @ok300

@kingonly
Copy link
Member Author

The payment was definitely successful in my case.
Regardless, I think we should just remove this message from the UI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants