-
Notifications
You must be signed in to change notification settings - Fork 65
apollo_dashboard: rename state sync stuck long time alert #12430
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
apollo_dashboard: rename state sync stuck long time alert #12430
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
3f99637 to
3f85fd0
Compare
09444c4 to
0c00495
Compare
yonatanw-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yonatanw-starkware reviewed 2 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).
3f85fd0 to
8f6de09
Compare
Merge activity
|
yonatanw-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@yonatanw-starkware made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).
Itay-Tsabary-Starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Itay-Tsabary-Starkware reviewed 2 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).
Itay-Tsabary-Starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Itay-Tsabary-Starkware made 1 comment.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).
idan-starkware
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@idan-starkware reviewed 2 files and all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).
66c8d98

Note
Low Risk
Naming/title changes only; alert expressions and thresholds are unchanged, with the main risk being minor dashboard/alert identifier mismatches if any external tooling depends on the previous titles.
Overview
Renames the state sync “stuck” alerts to use title-cased, more descriptive names, specifically distinguishing the long-duration variant as “State Sync Stuck Long Time”.
Updates
get_state_sync_stuck_vec()to pass human-readable titles while deriving the underlying alertnameby lowercasing and underscore-normalizing the title, and aligns the dev Grafana alert JSON titles accordingly.Written by Cursor Bugbot for commit 8f6de09. This will update automatically on new commits. Configure here.