Skip to content

feat(eap): Allow 30 day timeseries granularity - #130428

Closed
sentry-junior[bot] wants to merge 2 commits into
masterfrom
dominikbuszowiecki/eng-9009-30d-eap-granularity
Closed

sentry-junior[bot] wants to merge 2 commits into
masterfrom
dominikbuszowiecki/eng-9009-30d-eap-granularity

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Part of ENG-9009, which adds a 30 day interval option for 90 day dashboard charts.

  • Adds 30 days to VALID_GRANULARITIES. This lets spans, logs, trace metrics, transactions (spans) and mobile app size widgets request interval=30d, which validate_granularity currently rejects. Do not merge until feat(eap): Allow 30 day granularity for time series requests snuba#8686 is deployed, because Snuba's TimeSeries RPC has the same allowlist.
  • Keeps the other users of that set where they were:
    • EAP alert time windows stay capped at one day. Snuba subscriptions reject windows over 24h.
    • Uptime stats still defaults to at most a 1 day rollup when no resolution is passed. It used to fall back to max(VALID_GRANULARITIES), which would now be 30 days.
  • Fixes the issues timeseries meta.interval. It used timedelta.seconds, which drops whole days, so 1d and longer intervals reported 0.

Errors (legacy SnQL) and issues already accept a 30d interval. Release health still rejects it (sessions_v2 and the metrics layer only allow intervals that divide a day), and that will be a separate PR.

via @DominikB2014.

--

View Session [Sentry]

sentry-junior Bot and others added 2 commits November 9, 2026 14:38
Co-Authored-By: Dominik Buszowiecki <44422760+DominikB2014@users.noreply.github.com>
Co-Authored-By: Dominik Buszowiecki <44422760+DominikB2014@users.noreply.github.com>
@linear-code

linear-code Bot commented Nov 9, 2026

Copy link
Copy Markdown

ENG-9009

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant