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

server does not support the reflection API when celestia-appd starts with broken config #3858

Open
rootulp opened this issue Sep 11, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@rootulp
Copy link
Collaborator

rootulp commented Sep 11, 2024

Context

@renaynay and @cristaloleg had a validator with a broken config (app.toml, config.toml)

Problem

The broken config resulted in this symptom:

$ grpcurl -plaintext -d '{"subspace": "gov", "key": "votingparams"}' 51.159.158.12:9090 cosmos.params.v1beta1.Query/Params
Error invoking method "cosmos.params.v1beta1.Query/Params": failed to query for service descriptor "cosmos.params.v1beta1.Query": server does not support the reflection API

Basically all GRPC queries to the server failed with something like: server does not support the reflection API.

Proposal

  1. Get the broken configs from their validator so that we can repro the issue. I looked but couldn't find them on the Scaleway instance. cc: @cmwaters
  2. When celestia-app starts up, if it can't register reflection on the GRPC server, panic with an error message stating that config is invalid instead of starting up.
@rootulp rootulp added the bug Something isn't working label Sep 11, 2024
@rootulp
Copy link
Collaborator Author

rootulp commented Sep 11, 2024

The config on that node looks deleted. @evan-forbes reproed the issue with different config. Any chance you still have those and can attach them here?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant