Skip to content

Commit 709a139

Browse files
authored
Merge pull request #848 from fabriziosestito/fix/setup-metrics
fix: call set_meter_provider
2 parents ec6abaf + 1e7e344 commit 709a139

File tree

2 files changed

+10
-6
lines changed

2 files changed

+10
-6
lines changed

hack/docker-compose.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ services:
66
- "16686:16686"
77
- "4318:4317"
88
otel-collector:
9-
image: otel/opentelemetry-collector:0.98.0
9+
image: otel/opentelemetry-collector:0.103.0
1010
volumes:
1111
- ./otel-collector-minimal-config.yml:/etc/otel-collector-config.yml
1212
ports:

src/metrics.rs

+9-5
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
use opentelemetry::{metrics, KeyValue};
1+
use anyhow::Result;
2+
use opentelemetry::{global, KeyValue};
23
use opentelemetry_otlp::{ExportConfig, WithExportConfig};
3-
use opentelemetry_sdk::metrics::SdkMeterProvider;
44
use opentelemetry_sdk::runtime;
55

66
mod policy_evaluations_total;
@@ -10,15 +10,19 @@ pub use policy_evaluations_latency::record_policy_latency;
1010

1111
const METER_NAME: &str = "kubewarden";
1212

13-
pub fn setup_metrics() -> metrics::Result<SdkMeterProvider> {
14-
opentelemetry_otlp::new_pipeline()
13+
pub fn setup_metrics() -> Result<()> {
14+
let meter_provider = opentelemetry_otlp::new_pipeline()
1515
.metrics(runtime::Tokio)
1616
.with_exporter(
1717
opentelemetry_otlp::new_exporter()
1818
.tonic()
1919
.with_export_config(ExportConfig::default()),
2020
)
21-
.build()
21+
.build()?;
22+
23+
global::set_meter_provider(meter_provider.clone());
24+
25+
Ok(())
2226
}
2327

2428
pub trait PolicyEvaluationMetric: Into<Vec<KeyValue>> {}

0 commit comments

Comments
 (0)