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

Umbrel v0.5 & CLN app: avoid editing non-persisting files #8

Open
blckbx opened this issue Jul 14, 2022 · 1 comment
Open

Umbrel v0.5 & CLN app: avoid editing non-persisting files #8

blckbx opened this issue Jul 14, 2022 · 1 comment
Assignees
Labels
needs investigation Further investigation is required

Comments

@blckbx
Copy link
Collaborator

blckbx commented Jul 14, 2022

On Umbrel 0.5 we currently make changes to files that are being replaced on app updates (e.g. new cln version from app store). The changes need to be redone by the user each time which is not sustainable.
Instead of editing exports.sh and docker-compose.yml manually (see https://github.com/blckbx/tunnelsats#cln), we can create a default config file for CLN (Umbrel does not create that default config file on its own).

path to file:

~/umbrel/app-data/core-lightning/data/lightningd/config

contents:

bind-addr=0.0.0.0:9735
always-use-proxy=false
announce-addr={vpnIP}:{ vpnPort}

Problem here, we cannot use bind-addr in config file because it's already used by CLN's docker-compose.yml:

      - --bind-addr=${APP_CORE_LIGHTNING_DAEMON_IP}:9735

which is exposed as port 9736:

      - ${APP_CORE_LIGHTNING_DAEMON_PORT}:9735
...
export.sh:
      export APP_CORE_LIGHTNING_DAEMON_PORT="9736"

=> double-binding of port 9735

@blckbx blckbx changed the title Umbrel v0.5 & CLN app: avoid making changes to non-persisting files Umbrel v0.5 & CLN app: avoid editing non-persisting files Jul 14, 2022
@blckbx
Copy link
Collaborator Author

blckbx commented Jul 27, 2022

Making use of TunnelSats docker container IP, we can cut editing down to one file (port change of exports.sh) besides creating a separate CLN config file (which should be default on Umbrel).

bind-addr=10.9.9.9:9735
always-use-proxy=false
announce-addr={vpnIP}:{vpnPort}

@blckbx blckbx self-assigned this Sep 9, 2022
@blckbx blckbx added the needs investigation Further investigation is required label Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs investigation Further investigation is required
Projects
None yet
Development

No branches or pull requests

1 participant