mediacloud / web-search

Code that drives the public web-based tools for the Media Cloud Online News Archive and Directory.
https://search.mediacloud.org
Apache License 2.0
8 stars 12 forks source link

Distributed Tracing in Sentry #664

Open pgulley opened 2 weeks ago

pgulley commented 2 weeks ago

655 changes a setting to get tracing working correctly in the backend.

We might want to have some more configurability on the tracing/profiling rates- via more environment variables, as in news-search-api? Also wondering about configurability of frontend tracing- right now the SENTRY_DSN and related sentry init in react is all hard-coded.

pgulley commented 4 days ago

As far as configurability goes: the sentry initialization happens in root.jsx, which doesn't receive template varables from the backend. One high-priority easy change would be to move that around.

The other piece of this is to investigate if we want more intentional instrumentation, to limit the scope of the tracing spans we use- call that a medium priority