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

make more channels for amp #33

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

antonilol
Copy link

@antonilol antonilol commented Apr 19, 2022

now the alice container makes a channel to bob and carol of 1 mil sats each

this PR enables amp on container a b and c and makes 1 channel from each 3 to another (imagine a triangle)

stakwork/sphinx-relay#362 needs this patch and is now pointed to by fork but can be set back upstream once this is merged

https://github.com/antonilol/sphinx-relay/blob/amp/.github/workflows/integration_test.yml#L27

lnd/setup/index.js Outdated Show resolved Hide resolved
@antonilol
Copy link
Author

antonilol commented Apr 19, 2022

when the docker thing tries to open the bob channel to carol it cant find the macaroon

no such file or directory, open '/bob/.lnd/data/chain/bitcoin/regtest/admin.macaroon'

edit: oh wait i got it

@antonilol antonilol requested a review from kevkevinpal April 19, 2022 18:53
@antonilol
Copy link
Author

stakwork/sphinx-relay#362 (comment)

disable amp in the carol container?

lnd/setup/carol.conf Outdated Show resolved Hide resolved
@antonilol
Copy link
Author

maybe add dave without keysend but with amp. the standard 3 sat payments will go with amp then

@kevkevinpal
Copy link
Contributor

maybe add dave without keysend but with amp. the standard 3 sat payments will go with amp then

Yea this might be a good scenario to test aswell since newer nodes might not even have amp enabled

@antonilol
Copy link
Author

i tried to add dave but it doesnt seem to work well. the lnd starts but ask for a password and relay gives errors => initGrpcSubscriptions error cannot read LND macaroon or cert

@antonilol antonilol force-pushed the amp branch 2 times, most recently from ac27918 to 04ec760 Compare June 23, 2022 10:49
@antonilol antonilol force-pushed the amp branch 2 times, most recently from ccf5435 to d71ddc0 Compare September 16, 2022 20:31
@antonilol antonilol force-pushed the amp branch 11 times, most recently from 2a2d900 to d389a31 Compare October 2, 2022 13:43
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 this pull request may close these issues.

2 participants