Skip to content
This repository was archived by the owner on Jul 19, 2023. It is now read-only.

Add block opened metrics #785

Merged
merged 3 commits into from
Jun 22, 2023
Merged

Add block opened metrics #785

merged 3 commits into from
Jun 22, 2023

Conversation

cyriltovena
Copy link
Collaborator

This add the number of block currently opened and also the total samples in memory.

Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, apart from that one metric name

Co-authored-by: Christian Simon <[email protected]>
@cyriltovena cyriltovena merged commit 7e716d7 into main Jun 22, 2023
@cyriltovena cyriltovena deleted the feat/metrics_block_open branch June 22, 2023 21:35
simonswine added a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
* Add block opened metrics

* Add fake metrics to tests to fix it

* Update pkg/phlaredb/metrics.go

Co-authored-by: Christian Simon <[email protected]>

---------

Co-authored-by: Christian Simon <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants