Skip to content

fix(workflow): Remove noisy DetectorGroup.DoesNotExist log - #121024

Open
sentry[bot] wants to merge 1 commit into
masterfrom
seer/fix-detector-group-log-level-IKLjTi
Open

fix(workflow): Remove noisy DetectorGroup.DoesNotExist log#121024
sentry[bot] wants to merge 1 commit into
masterfrom
seer/fix-detector-group-log-level-IKLjTi

Conversation

@sentry

@sentry sentry Bot commented Jul 31, 2026

Copy link
Copy Markdown
Contributor
The `_get_detector_for_group` function in `sentry/workflow_engine/processors/detector.py` was logging `DetectorGroup.DoesNotExist` as an exception. This `DoesNotExist` is an expected fallback path when a group does not have a specific `DetectorGroup` associated with it, and the system correctly falls back to project-level or issue stream detectors.

Logging this as an exception was generating a large volume of non-actionable error events in Sentry. This change removes the log statement entirely, as the absence of a DetectorGroup is a normal and handled condition, not an error.

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. and is gonna need some rights from me in order to utilize my contributions in this here PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

Fixes SENTRY-5R4N

Comment @sentry <feedback> on this PR to have Autofix iterate on the changes.

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Jul 31, 2026
@kcons
kcons marked this pull request as ready for review August 3, 2026 18:48
@kcons
kcons requested a review from a team as a code owner August 3, 2026 18:48
@kcons
kcons self-requested a review August 3, 2026 18:48

@saponifi3d saponifi3d left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm - but i'd also be okay with us just removing this log tbh.

@sentry sentry Bot changed the title fix(workflow-engine): Reduce Sentry noise from DetectorGroup.DoesNotExist fix(workflow): Remove noisy DetectorGroup.DoesNotExist log Aug 3, 2026
@sentry
sentry Bot force-pushed the seer/fix-detector-group-log-level-IKLjTi branch from 22c0597 to 91ef389 Compare August 3, 2026 20:02
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