fedimint / fedimint

Federated E-Cash Mint
https://fedimint.org/
MIT License
536 stars 209 forks source link

Gateway liquidity api can list active channels #4776

Closed tvolk131 closed 2 weeks ago

tvolk131 commented 1 month ago

Requires #4514

Add lightning list-active-channels command to gateway CLI

elsirion commented 1 month ago

There seems to be a problem with one of the tests, I restarted CI but probably needs further investigation. Maybe rebasing on master where it doesn't fail could help.

bradleystachurski commented 1 month ago

@tvolk131 re: the CI flakiness from today's review club, looks like we're not giving enough time for lnd to startup. Hopefully fixed with https://github.com/fedimint/fedimint/pull/4984

tvolk131 commented 2 weeks ago

Needs rebase :(

Rebased!

tvolk131 commented 2 weeks ago

I believe you mentioned during a review club meeting that {inbound,outbound} liquidity isn't identical between CLN and LND due to node implementation details, but want to verify I'm not hallucinating that memory 😄

Correct! CLN liquidity data behaves strangely

bradleystachurski commented 1 week ago

review club: would be useful if we can capture a screenshot of the cln estimates to show the cln team