getsentry / getsentry/sentry

Take latency into consideration before regressing an issue

Open
#85,316 2 comments 1 reaction 0 assignees View on GitHub
Bug Product Area: Issues
Dominant language
Python
Stars
44.8k
Forks
4.9k
Avg merge
22h 21m
Merged PRs (30d)
586

Description

### Problem Statement

When resolving an issue (no release or commit conditions), if that issue receives a new error, it will always be changed into a regression, even if the event received is of high latency.

Customer comment:

> This issue was wrongly marked as a regression when an event with high latency was received.
>
> I would expect the logic to take into account the time when an issue was resolved and the latency when it receives events that were before that time, so it does not mark false positive regressions.

In this case, an issue regarding a mobile error was resolved at 12:04. However, an error that occurred at 08:48 was received at 12:55 due to high latency. This error regressed the issue, even though it actually occurred before the issue was resolved.

### Solution Brainstorm

Take latency into consideration and don't regress issues that receive events that occurred before it was resolved, even if those events are received after it was resolved.

### Product Area

Issues

Contributor guide

Open the contributing guide

Research direction

Start by tracing the issue resolution and regression logic, then inspect how event occurrence time and receipt time are compared. Reproduce the case where an event occurring before resolution arrives afterward; done means it does not regress the issue, while an event occurring after resolution still does.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, observability
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.