fix(cogs): Only attribute rate limit buckets to spans and transaction…

… metrics (#3622) Only spans and transactions are actually rate limited in that stage.
getsentry · May 17, 2024 · 826bfed · 826bfed
1 parent d61226f
commit 826bfed
Showing 1 changed file with 9 additions and 1 deletion.
diff --git a/relay-server/src/services/processor.rs b/relay-server/src/services/processor.rs
@@ -2671,7 +2671,15 @@ impl EnvelopeProcessorService {
             EnvelopeProcessor::SubmitClientReports(_) => AppFeature::ClientReports.into(),
             #[cfg(feature = "processing")]
             EnvelopeProcessor::RateLimitBuckets(v) => {
-                relay_metrics::cogs::ByCount(v.bucket_limiter.buckets()).into()
+                relay_metrics::cogs::ByCount(v.bucket_limiter.buckets().filter(|b| {
+                    // Only spans and transactions are actually rate limited at this point.
+                    // Other metrics do not cause costs.
+                    matches!(
+                        b.name.try_namespace(),
+                        Some(MetricNamespace::Spans | MetricNamespace::Transactions)
+                    )
+                }))
+                .into()
             }
         }
     }