feat(spans): disable redis span metrics extraction (#3600)

We originally re-enabled db.redis in order to collect metrics for the cache module. However, since then we decided that the cache module will only look at `cache.*` span ops. Therefore we can stop collecting `db.redis` span metrics.
getsentry · May 16, 2024 · fb06249 · fb06249
1 parent 25d12f4
commit fb06249
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 247 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -34,6 +34,7 @@
 - Restrict usage of OTel endpoint. ([#3597](github.com/getsentry/relay/pull/3597))
 - Support new cache span ops in metrics and tag extraction. ([#3598](https://github.com/getsentry/relay/pull/3598))
 - Extract additional user fields for spans. ([#3599](https://github.com/getsentry/relay/pull/3599))
+- Disable `db.redis` span metrics extraction. ([#3600](https://github.com/getsentry/relay/pull/3600))
 
 ## 24.4.2
 

diff --git a/relay-dynamic-config/src/defaults.rs b/relay-dynamic-config/src/defaults.rs
@@ -7,7 +7,13 @@ use crate::metrics::MetricSpec;
 use crate::{Feature, MetricExtractionConfig, ProjectConfig, Tag};
 
 /// A list of `span.op` patterns that indicate databases that should be skipped.
-const DISABLED_DATABASES: &[&str] = &["*clickhouse*", "*compile*", "*mongodb*", "db.orm"];
+const DISABLED_DATABASES: &[&str] = &[
+    "*clickhouse*",
+    "*compile*",
+    "*mongodb*",
+    "*redis*",
+    "db.orm",
+];
 
 /// A list of `span.op` patterns we want to enable for mobile.
 const MOBILE_OPS: &[&str] = &[