From 5244b0e3f778512406166f8946a80eeeef823e01 Mon Sep 17 00:00:00 2001 From: harsh kumar <135993950+hxrshxz@users.noreply.github.com> Date: Mon, 13 Oct 2025 00:50:58 +0530 Subject: [PATCH] Update scrape/manager_test.go Co-authored-by: George Krajcsovits Signed-off-by: harsh kumar <135993950+hxrshxz@users.noreply.github.com> --- scrape/manager_test.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/scrape/manager_test.go b/scrape/manager_test.go index f4dda1b5b8..0ee1549eb3 100644 --- a/scrape/manager_test.go +++ b/scrape/manager_test.go @@ -1175,9 +1175,7 @@ scrape_configs: require.Equal(t, histogram.Histogram{}, *got[0].h, "first sample should be zero sample") require.InDelta(t, expectedHistogramSum, got[1].h.Sum, 1e-9, "second sample should retain the expected sum") - // Note: Exemplars from classic histogram buckets are not preserved when converting to NHCB - // because the bucket series are replaced with a single native histogram sample. This is expected - // behavior. The test verifies that the presence of exemplars in the input doesn't cause errors. + require.Len(t, app.resultExemplars, 2, "expected 2 exemplars from histogram buckets") // The test successfully completes, proving that both ConvertClassicHistogramsToNHCBEnabled // and EnableCreatedTimestampZeroIngestion can work together without the error that was