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

ref(relay): Remove transaction metrics allowlist and bump relay version #48428

Merged
merged 20 commits into from
May 10, 2023

Conversation

TBS1996
Copy link
Contributor

@TBS1996 TBS1996 commented May 3, 2023

Now that we have removed the transaction metrics allowlist from relay (getsentry/relay#2092) we want to also remove it from sentry.

This also requires bumping the relay library version to 0.8.22 to pass project config validation tests.

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label May 3, 2023
@codecov
Copy link

codecov bot commented May 10, 2023

Codecov Report

Merging #48428 (3ef0ab3) into master (45ab91c) will decrease coverage by 0.01%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #48428      +/-   ##
==========================================
- Coverage   80.95%   80.95%   -0.01%     
==========================================
  Files        4817     4817              
  Lines      201922   201914       -8     
  Branches    11436    11436              
==========================================
- Hits       163471   163462       -9     
- Misses      38196    38197       +1     
  Partials      255      255              
Impacted Files Coverage Δ
src/sentry/relay/config/__init__.py 96.63% <ø> (-0.11%) ⬇️

... and 1 file with indirect coverage changes

@jjbayer jjbayer marked this pull request as ready for review May 10, 2023 12:55
@jjbayer jjbayer requested a review from a team as a code owner May 10, 2023 12:55
@jjbayer jjbayer requested a review from a team May 10, 2023 12:55
@jjbayer jjbayer changed the title ref(relay): Remove transaction metrics allowlist ref(relay): Remove transaction metrics allowlist and bump relay version May 10, 2023
@jjbayer jjbayer merged commit 9f54c32 into master May 10, 2023
@jjbayer jjbayer deleted the removeallowlist branch May 10, 2023 13:45
@github-actions github-actions bot locked and limited conversation to collaborators May 26, 2023
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