Skip to content

Conversation

shellmayr
Copy link
Member

@shellmayr shellmayr commented Oct 10, 2025

  • Use the context manager introduced in fix(aci): Quiet Redis exception noise #99955 to quiet redis noise in dynamic sampling
  • Move the context manager and related utils to a central point, since this will be useful across the repo

Closes TET-1227

@shellmayr shellmayr requested a review from a team as a code owner October 10, 2025 11:45
Copy link

linear bot commented Oct 10, 2025

@shellmayr shellmayr requested a review from a team October 10, 2025 11:45
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Oct 10, 2025
cursor[bot]

This comment was marked as outdated.

from sentry.taskworker.namespaces import relay_tasks
from sentry.utils import metrics
from sentry.utils.sdk import set_current_event_project
from sentry.workflow_engine.utils.sentry_level_utils import quiet_redis_noise
Copy link
Member

Choose a reason for hiding this comment

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

since this will now be used across sentry, this would be good to move to some shared utils

Copy link
Member Author

Choose a reason for hiding this comment

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

True! cc @kcons

Copy link
Member Author

Choose a reason for hiding this comment

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

@vgrozdanic moved it in this PR - but will be waiting for a review from Kyle to sign off on this, don't want to move this around without communicating it, as it's a quite recent addition

Copy link

codecov bot commented Oct 10, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

Additional details and impacted files
@@           Coverage Diff            @@
##           master   #101331   +/-   ##
========================================
  Coverage   81.04%    81.04%           
========================================
  Files        8699      8698    -1     
  Lines      385838    385831    -7     
  Branches    24400     24400           
========================================
+ Hits       312703    312707    +4     
+ Misses      72784     72773   -11     
  Partials      351       351           

@shellmayr shellmayr force-pushed the shellmayr/chore/quiet-redis-noise-dynamic-sampling branch from c594d3b to 9c9f9ed Compare October 13, 2025 07:48
Copy link
Member

@kcons kcons left a comment

Choose a reason for hiding this comment

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

The move lgtm.

@shellmayr shellmayr merged commit 06aedf6 into master Oct 15, 2025
65 checks passed
@shellmayr shellmayr deleted the shellmayr/chore/quiet-redis-noise-dynamic-sampling branch October 15, 2025 07:41
@shellmayr shellmayr added the Trigger: Revert Add to a merged PR to revert it (skips CI) label Oct 15, 2025
@getsentry-bot
Copy link
Contributor

PR reverted: 9105dc9

getsentry-bot added a commit that referenced this pull request Oct 15, 2025
…date_project_config (#101331)"

This reverts commit 06aedf6.

Co-authored-by: shellmayr <6788060+shellmayr@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components Trigger: Revert Add to a merged PR to revert it (skips CI)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants