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

Mixin: Lower alerting threshold for BucketIndexNotUpdated alert #7879

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

56quarters
Copy link
Contributor

@56quarters 56quarters commented Apr 11, 2024

What this PR does

This change will cause us to page when a bucket index has not been updated by two compactor cleanup cycles (every 15 minutes) with a 5 minute buffer to avoid false-positives.

The update time of the index is checked when queriers perform queries so as soon as it exceeds the 1 hour max age, queries begin to fail. Thus we need to alert on the age of the index before it begins to fail queries.

Which issue(s) this PR fixes or relates to

N/A

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.

@56quarters 56quarters force-pushed the 56quarters/stale-bucket-index branch from ba0c98e to 67aa6af Compare April 11, 2024 15:46
This change will cause us to page when a bucket index has not been updated
by two compactor cleanup cycles (every 15 minutes) with a 5 minute buffer to
avoid false-positives.

The update time of the index is checked when queriers perform queries so as
soon as it exceeds the 1 hour max age, queries begin to fail. Thus we need
to alert on the age of the index before it begins to fail queries.

Signed-off-by: Nick Pillitteri <[email protected]>
@56quarters 56quarters force-pushed the 56quarters/stale-bucket-index branch from 67aa6af to 8f388d2 Compare April 11, 2024 15:48
@56quarters 56quarters marked this pull request as ready for review April 11, 2024 15:53
@56quarters 56quarters requested a review from a team as a code owner April 11, 2024 15:53
@56quarters 56quarters merged commit 6f5018c into main Apr 11, 2024
31 checks passed
@56quarters 56quarters deleted the 56quarters/stale-bucket-index branch April 11, 2024 16:36
56quarters added a commit that referenced this pull request Apr 11, 2024
Add potential mitigations for stale bucket index alerts to the runbook.

Related #7879

Signed-off-by: Nick Pillitteri <[email protected]>
56quarters added a commit that referenced this pull request Apr 11, 2024
…7883)

Add potential mitigations for stale bucket index alerts to the runbook.

Related #7879

Signed-off-by: Nick Pillitteri <[email protected]>
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

Successfully merging this pull request may close these issues.

2 participants