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

Run without chainNodes/privValAddr #191

Closed
activenodes opened this issue Aug 23, 2023 · 5 comments
Closed

Run without chainNodes/privValAddr #191

activenodes opened this issue Aug 23, 2023 · 5 comments
Milestone

Comments

@activenodes
Copy link

@agouin thank you for this explanation https://github.com/strangelove-ventures/horcrux/issues/100

What do you think of this "hybrid" setup ?

2 Datacenter (with in each: 1 sentry and 1 signer)
1 Spare Datacenter (with 1 signer and without sentry)

DC01
Sentry01 <-> Signer01 <->|to S. 2+3

DC02
Sentry02 <-> Signer02 <->|to S. 1+3

DC03
             Signer03 <->|to S. 1+2

this way each sentry is connected to its own local signer (latency 0) and there is an additional signer reachable p2p by the other signers when a DC is down.

I think it's the minimum configuration, but with high performance and full redundancy.

Signers connected to all sentries may cause connections cross like Sentry01<->Signer02 and Sentry02<->Signer01. Right?

So in the Signer03 we need to run without "chainNodes/privValAddr" but the result is:

Error: need to have chainNodes configured for priv-val connection

@activenodes
Copy link
Author

Otherwise we could make this dynamic:

const connRetrySec = 2

and insert a dummy IP.

Example:

chainNodes:
- privValAddr: tcp://127.0.0.1:1234
  connRetrySec: 99999

@agouin
Copy link
Member

agouin commented Aug 26, 2023

This would be a nice feature. We will get this in the next release

@agouin agouin added this to the v3.2.0 milestone Aug 26, 2023
@activenodes
Copy link
Author

This would be a nice feature. We will get this in the next release

Thank you!
I'm migrating from some TMKMS clusters to Horcrux, testing it in every way... great development

@activenodes
Copy link
Author

activenodes commented Nov 29, 2023

@agouin please remember this 🙏

Note. the original idea was to run a cosigner without "chainNodes/privValAddr", without changing connRetrySec in a variable

@agouin
Copy link
Member

agouin commented Nov 30, 2023

This is now possible in v3.2.0

@agouin agouin closed this as completed Nov 30, 2023
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

No branches or pull requests

2 participants