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

Log fetched bucket index timestamps #9896

Merged
merged 3 commits into from
Nov 13, 2024

Conversation

jhalterman
Copy link
Member

@jhalterman jhalterman commented Nov 13, 2024

What this PR does

Adds logging of bucket index timestamps (updatedAt) after they're fetched, in queriers and store gateways. This allows better understanding of which exact bucket index queriers and store gateways are working off of.

Which issue(s) this PR fixes or relates to

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

Adds logging of bucket index timestamps after they're fetched, in queriers and store gateways.
@jhalterman jhalterman force-pushed the log-block-index-metadata branch from 4c4a3ab to 31378b2 Compare November 13, 2024 22:19
@jhalterman jhalterman marked this pull request as ready for review November 13, 2024 22:19
@jhalterman jhalterman requested a review from a team as a code owner November 13, 2024 22:19
@jhalterman jhalterman force-pushed the log-block-index-metadata branch from c42669a to d3a6c92 Compare November 13, 2024 22:24
@jhalterman jhalterman merged commit 05cd9ca into grafana:main Nov 13, 2024
31 checks passed
@jhalterman jhalterman deleted the log-block-index-metadata branch November 13, 2024 23:05
jhalterman added a commit that referenced this pull request Nov 13, 2024
* Log fetched bucket index timestamps

Adds logging of bucket index timestamps after they're fetched, in queriers and store gateways.

(cherry picked from commit 05cd9ca)

Co-authored-by: Jonathan Halterman <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants