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

Add cross-reference URLs between Sentry events and CI runs #4324

Open
armcknight opened this issue Sep 10, 2024 · 5 comments
Open

Add cross-reference URLs between Sentry events and CI runs #4324

armcknight opened this issue Sep 10, 2024 · 5 comments

Comments

@armcknight
Copy link
Member

Description

Following on #4321, I discovered a need to have a URL I could click from the Sentry detail page for an e.g. issue to the CI run it came from, and vice-versa.

Sentry -> GitHub Action CI run:
If the URL for a CI run could be directly accessed from the yml config variables, or reconstructed from environment info, then we should put that URL in a tag on the scope when running UI tests.

GItHub Action CI run -> Sentry:
Likewise, it would be nice if we could somehow construct the Sentry URLs that will be set for the events transmitted in a UI test; we know the project and event IDs, I'm not sure if that's enough info to build the URL that will eventually point to the detail page in the Sentry web app. Then, we could write it to a file in the UI tests, and preserve them as artifacts, so we could click a link in the GItHub Actions UI to get to the event in the Sentry web app.

@armcknight armcknight changed the title Add crossreference URLs between Sentry events and CI runs Add cross-reference URLs between Sentry events and CI runs Sep 10, 2024
@philipphofmann
Copy link
Member

@armcknight, can you elaborate on why you need the link between Sentry and GH actions? I was talking to @brustolin today, and we both never look at the events in Sentry stemming from CI. For us it's just noise, and we even thought about disabling these events or routing them to a different DSN, so the sentry-cocoa project in sentry-sdks in Sentry only has the manually created events we create while testing the sample app.

@armcknight
Copy link
Member Author

I like the idea of injecting a different DSN for CI runs to cut down on the noise in sentry-sdks. It is exactly that noise I was trying to cut through to see something.

@philipphofmann
Copy link
Member

Then, let's go with the different DSN for CI runs.

@armcknight
Copy link
Member Author

Chiming back in to say it would make things like getting to the txns/profiles like we're looking at in #4377 (comment) much faster and less error prone!

@philipphofmann
Copy link
Member

⬆ that makes sense. So, both extra DSN and cross-reference URLs would make sense.

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

No branches or pull requests

2 participants