ops: make sentry sampling rates configurable

Author: matt-codecov

objectstore-server/src/config.rs

@@ -31,6 +31,13 @@ pub enum Storage {
     },
 }
 
+#[derive(Debug, Clone, Deserialize, Serialize)]
+pub struct Sentry {
+    pub dsn: String,
+    pub sample_rate: Option<f32>,
+    pub traces_sample_rate: Option<f32>,
+}
+
 #[derive(Debug, Clone, Deserialize, Serialize)]
 pub struct Config {
     // server addr config
@@ -41,7 +48,7 @@ pub struct Config {
     pub long_term_storage: Storage,
 
     // others
-    pub sentry_dsn: Option<String>,
+    pub sentry: Option<Sentry>,
     pub datadog_key: Option<String>,
     pub metric_tags: BTreeMap<String, String>,
 }
@@ -58,7 +65,7 @@ impl Default for Config {
                 path: PathBuf::from("data"),
             },
 
-            sentry_dsn: None,
+            sentry: None,
             datadog_key: None,
             metric_tags: Default::default(),
         }

objectstore-server/src/observability.rs

@@ -1,9 +1,9 @@
 use std::env;
 
 use sentry::integrations::tracing as sentry_tracing;
-use tracing::Level;
 use tracing::level_filters::LevelFilter;
-use tracing_subscriber::{EnvFilter, prelude::*};
+use tracing::Level;
+use tracing_subscriber::{prelude::*, EnvFilter};
 
 use crate::config::Config;
 
@@ -22,12 +22,12 @@ pub fn maybe_initialize_metrics(config: &Config) -> std::io::Result<Option<merni
 }
 
 pub fn maybe_initialize_sentry(config: &Config) -> Option<sentry::ClientInitGuard> {
-    config.sentry_dsn.as_ref().map(|sentry_dsn| {
+    config.sentry.as_ref().map(|sentry_config| {
         sentry::init(sentry::ClientOptions {
-            dsn: sentry_dsn.parse().ok(),
+            dsn: sentry_config.dsn.parse().ok(),
             enable_logs: true,
-            sample_rate: 1.0,
-            traces_sample_rate: 1.0,
+            sample_rate: sentry_config.sample_rate.unwrap_or(1.0),
+            traces_sample_rate: sentry_config.traces_sample_rate.unwrap_or(0.01),
             ..Default::default()
         })
     })
@@ -36,7 +36,7 @@ pub fn maybe_initialize_sentry(config: &Config) -> Option<sentry::ClientInitGuar
 pub fn initialize_tracing(config: &Config) {
     // Same as the default filter, except it converts warnings into events
     // and also sends everything at or above INFO as logs instead of breadcrumbs.
-    let sentry_layer = config.sentry_dsn.as_ref().map(|_| {
+    let sentry_layer = config.sentry.as_ref().map(|_| {
         sentry_tracing::layer().event_filter(|metadata| match *metadata.level() {
             Level::ERROR | Level::WARN => {
                 sentry_tracing::EventFilter::Event | sentry_tracing::EventFilter::Log