Skip to content

Commit a62af2c

Browse files
kamilogorekschew2381
authored andcommitted
ref(sourcemaps): Use all events traffic as the base for sample rate (#46929)
Make sample rate work across all processed traffic, not only specific projects IDs.
1 parent 9f118af commit a62af2c

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

src/sentry/lang/javascript/utils.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
import random
2+
13
from sentry import options
24

35
SYMBOLICATOR_SOURCEMAPS_PROJECTS_OPTION = "symbolicator.sourcemaps-processing-projects"
@@ -9,7 +11,7 @@ def should_use_symbolicator_for_sourcemaps(project_id: int) -> bool:
911
# 11276 - sentry/javascript project for forced dogfooding
1012
# settings.SENTRY_PROJECT - default project for all installations
1113
# settings.SENTRY_FRONTEND_PROJECT - configurable default frontend project
12-
if project_id in options.get(SYMBOLICATOR_SOURCEMAPS_PROJECTS_OPTION, []):
14+
if project_id in options.get(SYMBOLICATOR_SOURCEMAPS_PROJECTS_OPTION):
1315
return True
1416

15-
return project_id % 1000 < options.get(SYMBOLICATOR_SOURCEMAPS_SAMPLE_RATE_OPTION, 0.0) * 1000
17+
return options.get(SYMBOLICATOR_SOURCEMAPS_SAMPLE_RATE_OPTION) > random.random()

0 commit comments

Comments
 (0)