tikv / rust-prometheus

Prometheus instrumentation library for Rust applications
Apache License 2.0
1.06k stars 182 forks source link

Registering the same metric multiple times #495

Open sazzer opened 1 year ago

sazzer commented 1 year ago

I've written some Axum middleware to record the number of times each HTTP request is processed, and I'm hitting problems with actually recording the metrics.

What I've done so far is this:

pub async fn record_metrics<B>(
    registry: Extension<prometheus::Registry>,
    request: Request<B>,
    next: Next<B>,
) -> Response {
    let uri = request.uri().to_string();

    let response = next.run(request).await;

    let counter = CounterVec::new(Opts::new("http_request", "HTTP Requests"), &["uri", "status_code"]).unwrap();
    registry.register(Box::new(counter.clone())).unwrap();

    counter.with_label_values(&[&uri, response.status().as_str()]).inc();

    response
}

And the call to registry.register() returns AlreadyReg on all incoming requests after the first one.

Is there some easy way to get back the already registered metric so that I can increment that instead?

I've looked at calling registry.gather() to try and find it, but can't trivially work out how to find the correct entry in the returned set.

Cheers

sunng87 commented 3 months ago

Hitting the same situation with #525. We need to be able to get the registered metric from register function. The ideal usage is whenever I call register with legal input, it returns the registered metric, whenever it is just registered, or registered by other call.