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

add the cln-grpc plugin #3109

Merged
merged 4 commits into from
May 13, 2022
Merged

add the cln-grpc plugin #3109

merged 4 commits into from
May 13, 2022

Conversation

openoms
Copy link
Collaborator

@openoms openoms commented May 11, 2022

releated #3054

@openoms openoms changed the base branch from v1.7 to dev May 11, 2022 08:57
/home/bitcoin/${netprefix}cl-plugins-enabled/
#blitz.conf.sh set [key] [value] [?conffile] <noquotes>
/home/admin/config.scripts/blitz.conf.sh set grpc-port "7777" ${CLCONF} noquotes
/home/admin/config.scripts/blitz.conf.sh set ${netprefix}cln-grpc-port "7777"
Copy link
Collaborator Author

@openoms openoms May 11, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rootzoll question: is there a benefit to use the port number here or should it be just
${netprefix}cln-grpc=on|off in the raspiblitz.conf and in redis?

Also the port number is completely arbitrary, suggestions are welcome.
For example on the 4*3 phone keyboard GRPC is 4772.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mean its nice to have the port numer as an option - the prestart script should take care of always setting it number. Only point is that its a bit more fuzzy as testing against "on" to check if the service shoudl be active. So now we would test if NOT "" - but that should be basically fine.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

port numbner 4772 sounds good with me - nice idea with the GRPC phonekeyboard coding :)

@openoms
Copy link
Collaborator Author

openoms commented May 11, 2022

building the grpc-plugin takes a good few minutes. I suggest it should be part of the "fatpack" SDcard image and can be on by default just like the experimental-features of CLN.

@rootzoll
Copy link
Collaborator

Please merge if you see this PR as done. Intense testing will be done as part of RC1

@openoms openoms added this to the 1.8.0 Release milestone May 13, 2022
@openoms
Copy link
Collaborator Author

openoms commented May 13, 2022

The cln-grpc plugin will be the part of the image release as it will be built after CLN in the build_sdcard.sh. Meging to dev for testing.

@openoms openoms merged commit 539afe9 into raspiblitz:dev May 13, 2022
@openoms openoms deleted the cln-grpc branch May 13, 2022 12:06
openoms added a commit that referenced this pull request May 13, 2022
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.

None yet

2 participants