Skip to content

feat(dashboards): Track dashboard generation validation attempts#116502

Merged
DominikB2014 merged 1 commit into
masterfrom
dominikbuszowiecki/track-dashboard-generation-retry-metric
May 29, 2026
Merged

feat(dashboards): Track dashboard generation validation attempts#116502
DominikB2014 merged 1 commit into
masterfrom
dominikbuszowiecki/track-dashboard-generation-retry-metric

Conversation

@DominikB2014
Copy link
Copy Markdown
Contributor

@DominikB2014 DominikB2014 commented May 29, 2026

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 29, 2026
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 29, 2026

DAIN-1700

Emit a sentry_sdk distribution metric, dashboards.on_completion_hook.
generation_attempts, exactly once per dashboard generation when it
reaches a terminal outcome (pass, max_retries, no_artifact). The value
is the number of attempts (retry_count + 1) and tags carry status
(pass|fail), result (pass|max_retries|no_artifact), and last_layer
(pydantic|serializer) when result=max_retries.

Lets dashboards answer how many tries the model needs to generate a
valid dashboard, success rate over time, and which validator layer
the model gets stuck on.

_within_retry_budget split into _count_retries (pure counter) plus
an inline budget check at the two failure branches, so the attempt
count is available for the metric. The existing max_retries_reached
log is preserved alongside.

Co-Authored-By: Claude <noreply@anthropic.com>
@DominikB2014 DominikB2014 force-pushed the dominikbuszowiecki/track-dashboard-generation-retry-metric branch from 2d8a22d to 32823bb Compare May 29, 2026 19:19
@DominikB2014 DominikB2014 marked this pull request as ready for review May 29, 2026 19:20
@DominikB2014 DominikB2014 requested a review from a team as a code owner May 29, 2026 19:20
@DominikB2014 DominikB2014 requested a review from shruthilayaj May 29, 2026 19:22
Copy link
Copy Markdown
Member

@nikkikapadia nikkikapadia left a comment

Choose a reason for hiding this comment

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

lgtm

@DominikB2014 DominikB2014 merged commit 29fc97b into master May 29, 2026
63 checks passed
@DominikB2014 DominikB2014 deleted the dominikbuszowiecki/track-dashboard-generation-retry-metric branch May 29, 2026 19:39
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.

2 participants