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

List and reorganise Tempo HTTP API #632

Closed
annanay25 opened this issue Apr 6, 2021 · 6 comments · Fixed by #938
Closed

List and reorganise Tempo HTTP API #632

annanay25 opened this issue Apr 6, 2021 · 6 comments · Fixed by #938
Labels
good first issue Good for newcomers

Comments

@annanay25
Copy link
Contributor

Over time we have added a lot of endpoints to Tempo's HTTP API, including /metrics /shutdown /config /memberlist etc. While a few of these provide information that is useful in debugging and monitoring, there are others that are used to perform administrative tasks, for instance the shutdown one, and should probably not be defined at root (/). As we start working towards a stable release, we should attempt to

  • add an "admin" endpoint that lists all the endpoints exposed by the target module
  • decide on an API nomenclature and move the endpoints to locations that make sense in terms of functionality
@annanay25 annanay25 added good first issue Good for newcomers enhancement New feature or request and removed enhancement New feature or request labels Apr 15, 2021
@tharun208
Copy link

@annanay25 I like to work on this

@annanay25
Copy link
Contributor Author

Its yours! @tharun208

@josephwoodward
Copy link
Contributor

@tharun208 Are you working on this? Otherwise I'm happy to pick it up.

@tharun208
Copy link

@josephwoodward yes, I am working on this

@josephwoodward
Copy link
Contributor

josephwoodward commented Jul 5, 2021

@tharun208 Okay, no problem 😁 Thanks for getting back to me.

@josephwoodward
Copy link
Contributor

@tharun208 How have you been getting on with this? Need a hand?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants