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

feat: add charts for a self-hosted signet setup #1125

Merged
merged 17 commits into from
Oct 3, 2022
Merged

feat: add charts for a self-hosted signet setup #1125

merged 17 commits into from
Oct 3, 2022

Conversation

openoms
Copy link
Contributor

@openoms openoms commented Jun 20, 2022

Discussed in: #1123

Depends on: GaloyMoney/blink#1385 (merged)

@openoms
Copy link
Contributor Author

openoms commented Jun 21, 2022

The exporter pod is failing:

exporter-5dbcc87b85-852bp                      0/1     CrashLoopBackOff   17 (106s ago)   70m
k3d@debian:~$ k -n galoy-dev-galoy logs exporter-5dbcc87b85-852bp
Defaulted container "exporter" out of: exporter, wait-for-mongodb-migrate (init)
/app/node_modules/bitcoin-core/dist/src/index.js:71
      throw new Error(`Invalid network name "${network}"`, {
      ^

Error: Invalid network name "signet"
    at new Client (/app/node_modules/bitcoin-core/dist/src/index.js:71:13)
    at new BitcoindClient (/app/lib/services/bitcoind/index.js:20:23)
    at Object.<anonymous> (/app/lib/services/bitcoind/index.js:87:33)
    at Module._compile (node:internal/modules/cjs/loader:1105:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Module.require (node:internal/modules/cjs/loader:1005:19)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at Module.Hook._require.Module.require (/app/node_modules/require-in-the-middle/index.js:80:39)
    at require (node:internal/modules/cjs/helpers:102:18)

@openoms openoms marked this pull request as ready for review June 25, 2022 15:53
@openoms openoms mentioned this pull request Jun 25, 2022
@openoms openoms force-pushed the dev-signet branch 4 times, most recently from 41a9265 to 7470c9b Compare September 20, 2022 10:18
@openoms
Copy link
Contributor Author

openoms commented Sep 20, 2022

Both the regtest and signet setup is functional locally and the Makefile is extended with new entries.

To test the signet deployment:

cd charts/dev
git checkout dev-signet
make all-signet

LND on signet generates the seed successfully, but seems to need one restart to have the networking active and switch the state from:
RPC_ACTIVE to SERVER_ACTIVE

Commands to test:

k -n galoy-sig-bitcoin  logs    lnd1-0 -c lnd -f
kubectl -n galoy-sig-bitcoin exec -it lnd1-0 -c lnd -- lncli -n signet state 
kubectl -n galoy-sig-bitcoin delete pod lnd1-0 --force

Edit: lnd does become active but needs a significantly longer time ~20 mins. Increasing the timeout for signet so it has time to sync fully.

Copy link
Member

@krtk6160 krtk6160 left a comment

Choose a reason for hiding this comment

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

Just have a comment regarding the docs, otherwise LGTM

dev/README.md Outdated Show resolved Hide resolved
@openoms openoms requested a review from krtk6160 September 29, 2022 07:06
@openoms openoms merged commit 20acf08 into main Oct 3, 2022
@openoms openoms deleted the dev-signet branch October 3, 2022 07:51
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