Clone Family.metrics and iterate over family about client_rust

[JaydenElliott]

Sure.

Iter

The primary problem I have found when using this library is that without knowing all the label keys, it's not possible (please let me know if it is) to get any metrics.

Suppose I have an application that requires me to spawn n components to process some workload. At runtime each of these will be spawned and assigned a uuid. In order to monitor these I would like to attach a metrics counter to each (using uuid as the primary label field). I will create a Family<ComponentLabel, Counter> for this.

Now suppose I am now in some other part of the application and would like to iterate over every metric in that family. The only way to do this at the moment (i believe) would be to store a vec of uuids that you inserted into the family and manually iterate over each and get_or_create().

The family "hashmap" is already storing all the data we need to iterate over it, but doesn't expose any functionality to interact with it (outside of get_or_create). Requiring a separate data structure to store insertions for later seems wasteful and unessesary.

Clone

Using the above example, suppose I want to compare metrics of each component over time. To do this, I would like to take a snapshot of the metric family at time t and then some time later in the processing take another snapshot and then compare the two. This may be used for example to see if the processing is lagging (counters are not increasing).

This is not possible at the moment as M (in `Family<S,M>) is not clone and the underlying family hashmap isn't exposed.

Suggestion

To solve both these problems I suggest making M clone (in Family<S,M>) and exposing the read method of family.metrics.

E.g. in metrics/family.rs:

 pub fn read(&self) -> RwLockReadGuard<HashMap<S, M>> {
        self.metrics.read()
    }

This would solve both these problems and allow a user to do something like:

family .read().iter() .map(|(label, counter)| { do_something() })

Also providing a snapshot() method on the registry,family or metrics would be nice. This should be easy to implement and give user's a nice interface to compare metrics over time.

[mxinden]

Can you expand on the use-cases that you want to solve with the above proposed changes?

[gamgi]

I am also looking for these features.

My use case is to expose some metrics and labels, via a proprietary endpoint in addition to the standard /metrics endpoint. Currently I have to store the inserted labels in a HashSet which seems unnecesary.