Skip to content

Add Sentry trace support#971

Merged
rgalanakis merged 6 commits intomainfrom
keepalive-false
Jun 26, 2025
Merged

Add Sentry trace support#971
rgalanakis merged 6 commits intomainfrom
keepalive-false

Conversation

@rgalanakis
Copy link
Contributor

Add Sentry trace support.

Here's the config info:

    # Baseline all other trace configurations refer to.
    # Turning this down will proportionately reduce all other traces.
    setting :traces_base_sample_rate, 0.1
    # Rate for most web requests, relative to base rate.
    setting :traces_web_sample_rate, 1
    # Rate for high-throughput "webhook" endpoints, like service integrations and 'install' calls,
    # relative to base rate.
    setting :traces_web_load_sample_rate, 0.05
    # Rate for most jobs, relative to base rate.
    setting :traces_job_sample_rate, 0.1
    # Rate for high-throughput process webhook jobs, relative to base rate.
    setting :traces_job_load_sample_rate, 0.05

See if we can narrow down the H27 errors in Heroku.
The H27s were just from massive new load
See if this gets rid of the queues.publish span we don't want
@codecov
Copy link

codecov bot commented Jun 25, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.63%. Comparing base (dbfdac7) to head (65fc18b).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #971      +/-   ##
==========================================
+ Coverage   97.62%   97.63%   +0.01%     
==========================================
  Files         490      490              
  Lines       31186    31269      +83     
==========================================
+ Hits        30444    30529      +85     
+ Misses        742      740       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rgalanakis rgalanakis merged commit 685c74d into main Jun 26, 2025
4 checks passed
@rgalanakis rgalanakis deleted the keepalive-false branch June 26, 2025 04:26
@rgalanakis rgalanakis changed the title Keepalive false Add Sentry trace support Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant