Histrogram/Series Panel about phlare HOT 6 CLOSED

I think we should also make sure that we make it pretty clear what the X-axis represents.

Because for histograms panel it will be time and for the flamegraph it is amount of resource used.

from phlare.

@joey-grafana has already tried the PR with the metrics API and was able to convert it into a data frame that gets rendered as a graph.

from phlare.

I'd like to request that the values should match up to data from other sources; e.g. "memory" for a Go program should match up to "go heap size" from Prometheus. (This might seem obvious, but it's not the case with Pyroscope).

from phlare.

@cyriltovena @simonswine None of the current profiles return multiple series for the time series graph. It would be great to see multiple series lines in the time series graph for testing.

Also, what can we label each of the series (lines) in the graph legend on the bottom of the graph?

Might relate to either of the following two;

• Select via dropdown labels name to group by (see the #160): I think this is what we're doing already when selecting profiles? Or that you'd like another dropdown to select labels..
• Add a new label selector based on a selected series in the query: I'd imagine if we select a particular series (which has a label), we can add that label in between the '{}' in the input field next to the profile label selector dropdown?

from phlare.

You need to pass the group by parameter to see multiple series. Otherwise it group them all into one series.

Select via dropdown labels name to group by (see the #160)

I'd like that we add a dropdown to select labels to group by, I was thinking inside the graph panel, but anywhere would do at first. This might requires a new LabelNames API ?

https://prometheus.io/docs/prometheus/latest/querying/api/#getting-label-names

from phlare.

I'd like to request that the values should match up to data from other sources; e.g. "memory" for a Go program should match up to "go heap size" from Prometheus. (This might seem obvious, but it's not the case with Pyroscope).

Can you expand on this request @bboreham

from phlare.