Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The number of crashes on the release page and the discover page differ significantly #70570

Open
PayneFuRC opened this issue May 9, 2024 · 5 comments

Comments

@PayneFuRC
Copy link

PayneFuRC commented May 9, 2024

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. There are 9.1k crashes in release page
    image

  2. There are 3.8k crashes in discover page
    image

  3. There's no dropped crashes

  4. There's no filter rules

Expected Result

These two data sets should be the same.

Actual Result

These two data sets differ significantly.

Product Area

Releases

Link

No response

DSN

No response

Version

No response

@getsantry
Copy link
Contributor

getsantry bot commented May 9, 2024

Assigning to @getsentry/support for routing ⏲️

@PayneFuRC
Copy link
Author

And the data from other versions is the opposite, the number of crashes on the release page is lower. If the logic of beforeSend filtered it out, this situation should not exist.
image (1)
image (2)

@getsantry
Copy link
Contributor

getsantry bot commented May 10, 2024

Routing to @getsentry/product-owners-releases for triage ⏲️

@nhsiehgit nhsiehgit self-assigned this May 10, 2024
@nhsiehgit
Copy link
Member

Hi @PayneFuRC

Thanks for the report!
It looks like our releases page may be deriving this count from somewhere unexpectedly different than our discover page.

I might err on trusting the discover query for the crash rate for now while we dig to understand where this discrepancy is stemming from.

will create an internal ticket for us to track

@nhsiehgit
Copy link
Member

Hi @PayneFuRC

Dug into this a bit more
So it looks like the releases page is filtering these events for 'session.status' === 'crashed' as opposed to simply error.unhandled:true

So the releases count actually will be the more proper count!
I'll work with the team to see how we can surface this in a more clear manner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

No branches or pull requests

3 participants