Skip to content

ref(ci): Do not send transactions from pytest-sentry#41640

Merged
jan-auer merged 4 commits intomasterfrom
ref/ci-pytest-no-traces
Nov 22, 2022
Merged

ref(ci): Do not send transactions from pytest-sentry#41640
jan-auer merged 4 commits intomasterfrom
ref/ci-pytest-no-traces

Conversation

@jan-auer
Copy link
Copy Markdown
Member

Our developers are not using transactions or tracing for tests right now, since we use pytest-sentry just to report flakey and broken tests.

Transactions sent from pytest-sentry frequently run into rate limits and create the second highest transaction volume in the sentry organization. Since we do not need them at all, we can disable them until there's more use for them.

@jan-auer jan-auer requested a review from a team as a code owner November 22, 2022 15:20
Copy link
Copy Markdown
Contributor

@asottile-sentry asottile-sentry left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jan-auer jan-auer merged commit f626a48 into master Nov 22, 2022
@jan-auer jan-auer deleted the ref/ci-pytest-no-traces branch November 22, 2022 15:56
@github-actions github-actions bot locked and limited conversation to collaborators Dec 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants