ENG-3832 chore(portal): Lower the Sentry sample rate to 0.5 #815
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ENG-3832 [Sentry] Lower the sample rate
Lower the sample rate to reduce logs and prevent rate limiting keegan: we can lower the sample rate more if needed on BE vital you may want to add a tech debt ticket to do the same. We should probably be fine with a sample rate of 0.5 or lower |
KeeganMyers
approved these changes
Aug 30, 2024
Vitalsine85
deleted the
vital/eng-0000/remove-chapter-3-incident-banner
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Affected Packages
Apps
Packages
Tools
Overview
This PR lowers the Sentry sample rate to 0.5 to help prevent hitting our limits. It also removes the Chapter 3 Incident banner as that issue has been resolved.
Screen Captures
If applicable, add screenshots or screen captures of your changes.
Declaration
PR Summary by Typo
Summary:
This pull request disables the production and staging banners and reduces Sentry's traces sample rate to 0.5 for both client-side and server-side.
Key Points:
.github/workflows/deploy_production.yml,.github/workflows/deploy_staging.yml,apps/portal/app/entry.client.tsx, andapps/portal/instrumentation.server.mjs.To turn off PR summary, please visit Notification settings.