Skip to content

Conversation

fionaliao
Copy link
Contributor

Part of #13531

Adding a few more basic tests for nhcb for <agg>_over_time, sum and avg and counter resets. The tests were inspired from similar tests for non-custom bucket native histograms in native_histograms.test.

Signed-off-by: Fiona Liao <fiona.liao@grafana.com>
@fionaliao fionaliao requested a review from krajorama October 3, 2024 19:23
krajorama
krajorama previously approved these changes Oct 8, 2024
Copy link
Member

@krajorama krajorama left a comment

Choose a reason for hiding this comment

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

Thank you, LGTM, might consider adding independent calculation from _bucket so it's more ease to check.

Co-authored-by: George Krajcsovits <krajorama@users.noreply.github.com>
Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com>
Signed-off-by: Fiona Liao <fiona.y.liao@gmail.com>
@krajorama krajorama merged commit 8650d25 into prometheus:main Oct 8, 2024
26 checks passed
julienduchesne pushed a commit to julienduchesne/prometheus that referenced this pull request Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants