Skip to content

Filter out Health Checks by Default and include Ping's #2227

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

Closed
thinkocapo opened this issue Jun 16, 2023 · 5 comments
Closed

Filter out Health Checks by Default and include Ping's #2227

thinkocapo opened this issue Jun 16, 2023 · 5 comments

Comments

@thinkocapo
Copy link

Problem Statement

Health checks will destroy your quota for transactions, whether you're in Free Tier or a Paid Plan. And if you're trialing then it's Unlimited so you don't care to filter them and you become a Paid Plan and get destroyed.

Solution Brainstorm

Asks.

  1. Filter out Ping's. According to this ticket, ping's are not filtered out and several commentators are advising it.
  2. Set it by default turned on, or else you will get destroyed and Sentry might not realize how many orgs this is happening to. Why not? Here's a Inbound Filter GH Issue, please make sure it filters out the Pings as well. As well as heartbeats.

Product Area

Performance

@getsantry
Copy link

getsantry bot commented Jun 16, 2023

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link

getsantry bot commented Jun 16, 2023

Routing to @getsentry/product-owners-performance for triage due by Monday, June 19 5:00 pm (yyz).⏲️

@brentc brentc transferred this issue from getsentry/sentry Jun 16, 2023
@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 Jun 16, 2023
@brentc
Copy link

brentc commented Jun 16, 2023

Transferring this over to the relay repo because I this is where it actually belongs. Let me know if there's a better place for it.

@jernejstrasner
Copy link

Seems like this is tracked elsewhere already (Telemetry Experience) and all Relay work has been done (or is also tracked there) so will close this.

@thinkocapo
Copy link
Author

Noting for myself this is the Telemetry Experience one which is open and I can track this from there, let me know if I got that wrong.
getsentry/sentry#46731

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

No branches or pull requests

4 participants