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

Start metrics server first before instantiating the rest of ingester #1370

Open
masih opened this issue Mar 7, 2023 · 0 comments
Open

Start metrics server first before instantiating the rest of ingester #1370

masih opened this issue Mar 7, 2023 · 0 comments

Comments

@masih
Copy link
Member

masih commented Mar 7, 2023

Admin server also exposes the debugging endpoints. In daemon, it is started after the ingester instantiation is finished. As a result if the code is slow to start we have no visibility into what's happening.

Refactor the code such that:

  • debugging endpoints are accessible as soon as possible and do not depend on initialization of heavy weight components
  • the node remains "unready" until all components are instantiated. This means the readiness check endpoint also exposed by the admin server should return non 200 until all components are initialised.
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

1 participant