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

Transaction Alerts | Transactions filtered for devices do not collect data #58753

Closed
andyl-ioki opened this issue Oct 25, 2023 · 15 comments · Fixed by #59995
Closed

Transaction Alerts | Transactions filtered for devices do not collect data #58753

andyl-ioki opened this issue Oct 25, 2023 · 15 comments · Fixed by #59995
Assignees

Comments

@andyl-ioki
Copy link

andyl-ioki commented Oct 25, 2023

Environment

SaaS (https://sentry.io/)

Steps to Reproduce

  1. Create a transaction with a specific device, e.g. device:SM-A226B
  2. Note the warning icon next to the filter input which states:

    We don't routinely collect metrics from "device". However, we do so once this alert has been saved.

  3. Save the alert.
  4. Check the alert and see in the Transaction Duration part that the "Summary" shows 100% for the clock icon - "The time spent waiting for metrics matching the filters used."

Note:
We made sure that the tag device:SM-A226B and is available for our project. So there are users with this specific device and transactions gathered with it.
We tried also other tags like app.device with the same result.

This is related to an issue I created earlier. Since then it only showed me the empty graph. I thought that it will change over time, but it does not. :(

Expected Result

After a short while the page shows data in the alert graph.
Alerts are triggered (if necessary) properly.

Actual Result

Transactions graph of the alert stays empty (for weeks).
Waiting time stays at 100%.
Screenshot 2023-10-25 at 12 00 49

When clicking on "Open in Discover" you can see a proper graph:
Screenshot 2023-10-25 at 12 02 48

When getting back to edit the rule, even there a proper data graph is shown, with a small information noting "The chart data above is an estimate based on the stored transactions that match the filters specified.":
Screenshot 2023-10-25 at 12 04 54

Note:
I created very low thresholds for this alert to test if at least they trigger, but they didn't. :(

Product Area

Alerts

Link

No response

DSN

No response

Version

No response

@getsantry
Copy link
Contributor

getsantry bot commented Oct 25, 2023

Assigning to @getsentry/support for routing ⏲️

@getsantry
Copy link
Contributor

getsantry bot commented Oct 25, 2023

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

@getsantry getsantry bot moved this from Waiting for: Support to Waiting for: Product Owner in GitHub Issues with 👀 Oct 25, 2023
@roggenkemper
Copy link
Member

thanks for bringing this up again! i've reached out to the team in charge of this project to get an answer for you.

@andyl-ioki
Copy link
Author

andyl-ioki commented Nov 13, 2023

We are receiving data now.
Thanks so much!

@andyl-ioki
Copy link
Author

andyl-ioki commented Nov 15, 2023

Unfortunately I have to reopen this issue again.
It was working for me at that time, when I wrote the comment above ☝️

But since then I did not have the chance to see the correct view again. It is like described in the issue again. No data at all and waiting time stays at 100%. :(

Maybe I switched your A/B Testing group and did not get back in there? :D

@roggenkemper can you check that again? Thanks in advance :)

@matejminar
Copy link
Member

Hey @andyl-ioki , this should be fixed as soon as getsentry/relay#2728 is deployed. 👍

@andyl-ioki
Copy link
Author

Cool. Thanks for the reply.

@matejminar
Copy link
Member

@andyl-ioki Can you confirm it's working now?

@andyl-ioki
Copy link
Author

Hej @matejminar,

All in all: No, it is not working :(

In detail:
Yesterday I checked the existing alert. Rechecked it today as well. I also modified the alert and checked it again after couple of hours. It is still not showing any graph/data. (When I edit the alert, it shows a striped graph with the info that the data is not yet collected but then after saving the modifications of the rule)

I also created a new rule today, with the same filters ... no luck. I also can't see the striped graph. It's just empty.

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 Nov 21, 2023
@matejminar
Copy link
Member

I'll have a chat with our ingest team. Everything else looks good from a product standpoint. Sorry for any inconvenience caused.

@andyl-ioki
Copy link
Author

Thanks for the fast reply and taking care of it 👍
I will let you know if something changes on our "Alert" side.

@obostjancic
Copy link
Member

Hi @andyl-ioki We have taken a closer look at this, and have a suspicion that it is related to the environment filter condition. In order to verify that, would you mind temporarily editing the alert to target "All environments" instead of the currently selected one. It should take a couple of minutes to propagate the change, but after that you should start seeing the data come in.

@andyl-ioki
Copy link
Author

@obostjancic I will have a look now 👀

@getsantry getsantry bot moved this to Waiting for: Product Owner in GitHub Issues with 👀 Nov 23, 2023
@andyl-ioki
Copy link
Author

@obostjancic
Sorry for my late reply.

I have good news - when I removed the filter for the environment the graph starts to show up 🎉
Screenshot 2023-11-24 at 06 41 55

I don't know if the warning and critical messages behave correctly, but we will see.

@obostjancic
Copy link
Member

@andyl-ioki Great! 🎉 The fix for environment filter should then land with #59995. I'll link this issue to it, so that you get notified when it's merged.

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

Successfully merging a pull request may close this issue.

6 participants