Skip to content

added command to connect to all peers of the ring of fire. - #21

Merged
RooSoft merged 2 commits into
RooSoft:mainfrom
ramondevries:main
Nov 21, 2022
Merged

added command to connect to all peers of the ring of fire.#21
RooSoft merged 2 commits into
RooSoft:mainfrom
ramondevries:main

Conversation

@ramondevries

Copy link
Copy Markdown
Collaborator

As a ring leader you want to be able to connect to all peers of the ring of fire to test connection of all peers.
The PR will add this command to the igniter script.

@RooSoft

RooSoft commented Nov 21, 2022

Copy link
Copy Markdown
Owner

@dsbaars are you equiped to test this awesome PR?

@dsbaars

dsbaars commented Nov 21, 2022 via email

Copy link
Copy Markdown
Collaborator

@RooSoft

RooSoft commented Nov 21, 2022

Copy link
Copy Markdown
Owner

I am on a long trip at least until Christmas so unfortunately not right now

Do you know anybody who can?

@Tinus016

Copy link
Copy Markdown

I am on a long trip at least until Christmas so unfortunately not right now

Do you know anybody who can?

I'm not @dsbaars but I can and I've tested it, and it worked (Raspiblitz V1.8)

Tested with the participants of the #SRROF_1Msats_53rdRING

Note: i have shorted the address dude privacy reasons.

a****@192.***.*.*:~/igniter(main) ₿ ./igniter.sh connect
Already connected to: 02ec23c7c0c1adb58a46f6ca5...
Connecting to: 034f0854c5b3233af6f24dacc38bf01d...
{

}
Connecting to: 02e9aa24976253137610291c127517592...
{

}
Connecting to: 0322ffef979b5977f644967968e1a3673f...
{

}
Connecting to: 03d608e2311bf91fe415a407e226632906...
{

}
Connecting to: 0290fe12d15c563e26af79e6eac924c92c4...
{

}
Already connected to: 03f55b4f277b85df749ce23e4f5525.....
a****@192.***.*.*:~/igniter(main) ₿ 

@RooSoft

RooSoft commented Nov 21, 2022

Copy link
Copy Markdown
Owner

Great, thanks guys for the contribution, will merge this right now.

@RooSoft
RooSoft merged commit f1a5bdf into RooSoft:main Nov 21, 2022
ramondevries added a commit to ramondevries/igniter that referenced this pull request Nov 21, 2022
Merge pull request RooSoft#21 from ramondevries/main
@ramondevries

Copy link
Copy Markdown
Collaborator Author

Today I took part in a ring balance in SRROF_1Msats_54thRING, the ringleader had a debian installation with docker and got several errors:

./igniter.sh: line 41: docker exec -i lnd lncli: command not found

Will need to investigate this.

Config of ring leader:

Debian GNU/Linux 10 (buster)
Docker version 20.10.21, build baeda1f
Docker Compose version v2.12.2
Running on Raspberry Pi 4 and all with docker compose

@RooSoft

RooSoft commented Nov 22, 2022

Copy link
Copy Markdown
Owner

Looks like the docker command is not available in the context of the current user.

From memory, default installations require sudo privileges to execute it.

@Tinus016

Tinus016 commented Nov 22, 2022

Copy link
Copy Markdown

Did he make the file executable? Like chmod u+x igniter.sh

@Tinus016

Copy link
Copy Markdown

Did he make the file executable? Like chmod u+x igniter.sh

Never mind..He did, otherwise he has no permissions to execute the file anyway

@RooSoft

RooSoft commented Nov 22, 2022

Copy link
Copy Markdown
Owner

Did he make the file executable? Like chmod u+x igniter.sh

Never mind..He did, otherwise he has no permissions to execute the file anyway

Yeah, it executes up until line 41, point at which it complains it doesn't know where the docker executable is.

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.

4 participants