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

CosmosFullNode should keep track of reference rpc heights. #325

Open
DavidNix opened this issue Jul 14, 2023 · 1 comment
Open

CosmosFullNode should keep track of reference rpc heights. #325

DavidNix opened this issue Jul 14, 2023 · 1 comment
Labels
proposal An idea that needs validation before starting work.

Comments

@DavidNix
Copy link
Contributor

DavidNix commented Jul 14, 2023

Right now, this only benefits the height drift mitigation feature where we can check ref rpc heights along with the pod heights. There may be ways to leverage this feature for other features, but I'm unsure of specific examples at this time.

However, this adds more complexity.

  • Ref data must be cached
  • Configuring ref rpcs complicates configuration as a whole.
  • Error handling as ref rpcs are can be flaky.
  • Automatically pulling ref rpc data from chain registry.
@DavidNix DavidNix added the proposal An idea that needs validation before starting work. label Sep 28, 2023
@DavidNix
Copy link
Contributor Author

I'm not sure if this is an important feature. It adds complexity and only powers height drift detection.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proposal An idea that needs validation before starting work.
Projects
None yet
Development

No branches or pull requests

1 participant