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

Add UI (e.g bucket web) to Compactor #1657

Closed
bwplotka opened this issue Oct 16, 2019 · 9 comments · Fixed by #1714
Closed

Add UI (e.g bucket web) to Compactor #1657

bwplotka opened this issue Oct 16, 2019 · 9 comments · Fixed by #1714

Comments

@bwplotka
Copy link
Member

I would like to propose an optional UI to the compactor, which will render what blocks we have access to same as bucket web.

AC:

  • Compactor having UI with similar view as thanos bucket web
@jojohappy
Copy link
Member

jojohappy commented Oct 17, 2019

What is the difference with thanos bucket web?

@squat
Copy link
Member

squat commented Oct 17, 2019

I think the main difference is that it runs as a daemon and you can access it whenever. I am a big fan of surfacing this UI easily ((((:

@bwplotka
Copy link
Member Author

Yes and everyone needs compactor anyway, so it would fit perfectly.

@squat
Copy link
Member

squat commented Oct 24, 2019

ack, I started hacking on this last week, I can put up a PR soon (:

@bwplotka
Copy link
Member Author

❤️

squat added a commit to squat/thanos that referenced this issue Oct 31, 2019
Currently, the bucket web command generates and registers metrics but
they are never actually exposed. This commit ensures that the metrics
are exposed and leverages the recently created server package for
consistency.

This cleanup also helps prepare for the upcoming changes for
thanos-io#1657.

Signed-off-by: Lucas Servén Marín <[email protected]>
squat added a commit to squat/thanos that referenced this issue Oct 31, 2019
Currently, the bucket web command generates and registers metrics but
they are never actually exposed. This commit ensures that the metrics
are exposed and leverages the recently created server package for
consistency.

This cleanup also helps prepare for the upcoming changes for
thanos-io#1657.

Signed-off-by: Lucas Servén Marín <[email protected]>
brancz pushed a commit that referenced this issue Oct 31, 2019
Currently, the bucket web command generates and registers metrics but
they are never actually exposed. This commit ensures that the metrics
are exposed and leverages the recently created server package for
consistency.

This cleanup also helps prepare for the upcoming changes for
#1657.

Signed-off-by: Lucas Servén Marín <[email protected]>
@stale
Copy link

stale bot commented Jan 11, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 11, 2020
@stale stale bot closed this as completed Jan 18, 2020
@bwplotka
Copy link
Member Author

Not yet done and in progress.

@stale
Copy link

stale bot commented Feb 27, 2020

This issue/PR has been automatically marked as stale because it has not had recent activity. Please comment on status otherwise the issue will be closed in a week. Thank you for your contributions.

@stale stale bot added the stale label Feb 27, 2020
@bwplotka
Copy link
Member Author

bwplotka commented Feb 27, 2020 via email

@stale stale bot removed the stale label Feb 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants