Skip to content

feat(symbolicator): Option comparing stackwalkers (flagr)#25812

Merged
loewenheim merged 2 commits into
masterfrom
feat/symbolicator-compare-stackwalking-flagr
May 11, 2021
Merged

feat(symbolicator): Option comparing stackwalkers (flagr)#25812
loewenheim merged 2 commits into
masterfrom
feat/symbolicator-compare-stackwalking-flagr

Conversation

@flub

@flub flub commented May 3, 2021

Copy link
Copy Markdown
Contributor

Symbolicator has a new per-request option to test a new stackwalker,
this sentry option allows us to turn this on for a small ratio of
the requests so we can assess the performance and quality impacts
without impacting normal production.

This version relies on Flagr only enabling this for a percentage
of the requests, see the flag on flagr:
https://flagr.getsentry.net/#/flags/67


See also the alternative without flagr: #25807

This also needs a PR in getsentry and needs the feature to land in symbolicator (PRs to be linked here when they exist)

@flub flub changed the title feat(symbolicator): Option comparing stackwalkers feat(symbolicator): Option comparing stackwalkers (flagr) May 3, 2021
@loewenheim loewenheim marked this pull request as ready for review May 5, 2021 14:20
Floris Bruynooghe and others added 2 commits May 6, 2021 10:45
Symbolicator has a new per-request option to test a new stackwalker,
this sentry option allows us to turn this on for a small ratio of
the requests so we can assess the performance and quality impacts
without impacting normal production.

This version relies on Flagr only enabling this for a percentage
of the requests, see the flag on flagr:
https://flagr.getsentry.net/#/flags/67
@loewenheim loewenheim merged commit 49442f0 into master May 11, 2021
@loewenheim loewenheim deleted the feat/symbolicator-compare-stackwalking-flagr branch May 11, 2021 11:49
@github-actions github-actions Bot locked and limited conversation to collaborators May 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants