Skip to content

Conversation

@Itay-Tsabary-Starkware
Copy link
Collaborator

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware commented Feb 9, 2026

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 alert name by 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.

@reviewable-StarkWare
Copy link

This change is Reviewable

Copy link
Collaborator Author

Itay-Tsabary-Starkware commented Feb 9, 2026

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware force-pushed the 02-09-apollo_dashboard_rename_state_sync_stuck_long_time_alert branch from 3f99637 to 3f85fd0 Compare February 9, 2026 15:04
@Itay-Tsabary-Starkware Itay-Tsabary-Starkware force-pushed the 02-08-apollo_dashboard_remove_duplicated_per-env_dashboard_files branch from 09444c4 to 0c00495 Compare February 9, 2026 15:04
Copy link

@yonatanw-starkware yonatanw-starkware left a 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: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware changed the base branch from 02-08-apollo_dashboard_remove_duplicated_per-env_dashboard_files to main-v0.14.1-committer February 9, 2026 18:32
@Itay-Tsabary-Starkware Itay-Tsabary-Starkware force-pushed the 02-09-apollo_dashboard_rename_state_sync_stuck_long_time_alert branch from 3f85fd0 to 8f6de09 Compare February 9, 2026 18:33
@graphite-app
Copy link

graphite-app bot commented Feb 9, 2026

Merge activity

  • Feb 9, 6:33 PM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

Copy link

@yonatanw-starkware yonatanw-starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

@yonatanw-starkware made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).

Copy link
Collaborator Author

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware left a 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: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).

Copy link
Collaborator Author

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

@Itay-Tsabary-Starkware made 1 comment.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).

Copy link
Contributor

@idan-starkware idan-starkware left a 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: :shipit: complete! all files reviewed, all discussions resolved (waiting on @Itay-Tsabary-Starkware).

@Itay-Tsabary-Starkware Itay-Tsabary-Starkware added this pull request to the merge queue Feb 10, 2026
Merged via the queue into main-v0.14.1-committer with commit 66c8d98 Feb 10, 2026
13 of 20 checks passed
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.

4 participants