Skip to content

apollo_dashboard: alert on frozen exchange-rate oracle (eth_to_strk, strk_to_usd) (#14706)#14764

Open
matanl-starkware wants to merge 1 commit into
matanl/bp-14703-strk-usd-oracle-alertsfrom
matanl/bp-14706-frozen-oracle-alert
Open

apollo_dashboard: alert on frozen exchange-rate oracle (eth_to_strk, strk_to_usd) (#14706)#14764
matanl-starkware wants to merge 1 commit into
matanl/bp-14703-strk-usd-oracle-alertsfrom
matanl/bp-14706-frozen-oracle-alert

Conversation

@matanl-starkware

@matanl-starkware matanl-starkware commented Jul 10, 2026

Copy link
Copy Markdown
Collaborator

Backport of #14706 to main-v0.14.3.

Clean git cherry-pick -x of a620b9af39. No starknet_version change (V0_14_3 preserved).

Stacked on matanl/bp-14703-strk-usd-oracle-alerts — merge that first.

Generated dev_grafana_alerts.json regenerated and verified identical (no-op).

…strk_to_usd) (#14706)

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
(cherry picked from commit e871c40)
@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@matanl-starkware matanl-starkware marked this pull request as ready for review July 10, 2026 21:42
@cursor

cursor Bot commented Jul 10, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Observability-only alert definitions; no runtime oracle or gas-pricing logic changes.

Overview
Adds frozen-feed monitoring for L1 gas price oracles alongside existing success/error alerts. eth_to_strk_rate_frozen and strk_to_usd_rate_frozen fire when sum(changes(<rate gauge>[1h])) stays below 1—i.e. the published rate did not move over an hour while the gauge is still present (no or vector(0), so missing metrics stay no-data).

Implementation introduces shared oracle_rate_frozen_alert on ETH_TO_STRK_RATE / SNIP35_STRK_USD_RATE, registers both in get_apollo_alerts, and mirrors them in dev_grafana_alerts.json with configurable severity placeholders. These alerts use ObserverApplicability::Applicable and omit the is_observer == 0 filter so observer nodes also surface a stale upstream price.

Reviewed by Cursor Bugbot for commit a620b9a. Bugbot is set up for automated code reviews on this repo. Configure here.

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