Just remove it from cargo

Author: scottgerring

opentelemetry-sdk/Cargo.toml

@@ -98,10 +98,13 @@ name = "batch_span_processor"
 harness = false
 required-features = ["rt-tokio", "testing"]
 
-[[bench]]
-name = "metric"
-harness = false
-required-features = ["metrics"]
+# TODO - this fails periodically on CI
+# e.g. https://github.com/open-telemetry/opentelemetry-rust/actions/runs/13538892896/job/37835456869
+# Work out why, and re-add.
+#[[bench]]
+#name = "metric"
+#harness = false
+#required-features = ["metrics"]
 
 [[bench]]
 name = "log"

opentelemetry-sdk/benches/metric.rs

@@ -338,16 +338,15 @@ fn benchmark_collect_histogram(b: &mut Bencher, n: usize) {
         h.record(1, &[]);
     }
 
-    let mut _rm = ResourceMetrics {
+    let mut rm = ResourceMetrics {
         resource: Resource::builder_empty().build(),
         scope_metrics: Vec::new(),
     };
 
-    // TODO - this assertion fails intermittently. Work out why!
-    // b.iter(|| {
-    //     let _ = r.collect(&mut rm);
-    //     assert_eq!(rm.scope_metrics[0].metrics.len(), n);
-    // })
+    b.iter(|| {
+        let _ = r.collect(&mut rm);
+        assert_eq!(rm.scope_metrics[0].metrics.len(), n);
+    })
 }
 
 criterion_group!(benches, counters, histograms);