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

Store: add data touched by type to the dashboard #5838

Conversation

douglascamata
Copy link
Contributor

@douglascamata douglascamata commented Oct 28, 2022

Signed-off-by: Douglas Camata 159076+douglascamata@users.noreply.github.com

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

  • Adds a data touched by type widget to the Thanos Store dashboard.
  • Adds the data_type dimensions to the data touched/fetched widgets to provide deeper insights.

Depends on #5819.

Verification

Tested queries in a local Grafana instance.

Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>
Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>
@douglascamata douglascamata marked this pull request as ready for review October 28, 2022 15:49
@douglascamata douglascamata marked this pull request as draft October 28, 2022 15:49
@douglascamata douglascamata marked this pull request as ready for review October 28, 2022 15:51
Copy link
Member

@saswatamcode saswatamcode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Copy link
Contributor

@philipgough philipgough left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Copy link
Member

@GiedriusS GiedriusS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💪 thanks!

@GiedriusS GiedriusS merged commit 46873aa into thanos-io:main Nov 25, 2022
douglascamata added a commit to rhobs/configuration that referenced this pull request Jan 11, 2023
Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>

This should include the following PRs to improve the Store dashboard, with the ultimate goal in helping out deciding values for its limits:

* thanos-io/thanos#5989
* thanos-io/thanos#5838
ngraham20 pushed a commit to ngraham20/thanos that referenced this pull request May 18, 2023
* Store: add data touched by type to the dashboard

Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>

* Fix captions for data fetched and touched widgets

Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>

Signed-off-by: Douglas Camata <159076+douglascamata@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants