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 sentry svc creation logic to connect with horcrux-proxy automatically #428

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

qj0r9j0vc2
Copy link

If set sentry mode, private validator will be horcrux what is connected through horcrux-proxy.

In horcrux-proxy, it has automatic discover service logic but cosmos-operator doesn't have automatic service creation logic.

through this improvement, horcrux, horcrux-proxy, and cosmos fullnode will be connected automatically.

If set sentry mode, private validator will be horcrux what is connected through horcrux-proxy. In horcrux-proxy, it has automatic discover service logic but cosmos-operator doesn't have automatic service creation logic.
through this improvement, horcrux, horcrux-proxy, and cosmos fullnode will be connected automatically.
@allthatjazzleo
Copy link
Contributor

this feature is wanted.
Currently, ones need to manually create k8s services for privval connection from horcrux-proxy

@agouin
Copy link
Member

agouin commented Oct 4, 2024

This is a great feature! We will want test cases before we can merge, but we will start testing this out in the next week or so.

@nourspace nourspace requested a review from agouin October 7, 2024 18:22
@nourspace nourspace requested a review from vimystic December 3, 2024 16:08
@nourspace
Copy link
Member

@vimystic please take it home

cc @agouin

vimystic and others added 2 commits December 16, 2024 01:35
Based on suggestion

Co-authored-by: Leo Pang <[email protected]>
Based on suggestion

Co-authored-by: Leo Pang <[email protected]>
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.

5 participants