diff --git a/CHANGELOG.md b/CHANGELOG.md index 605a68ed0bb..ea4f873b37d 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -7,7 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0 ## [Unreleased](https://github.com/open-telemetry/opentelemetry-python/compare/v1.13.0-0.34b0...HEAD) -## [1.13.0-0.34b0](https://github.com/open-telemetry/opentelemetry-python/releases/tag/v1.13.0-0.34b0) - 2022-09-26 +- Update explicit histogram bucket boundaries + ([#2947](https://github.com/open-telemetry/opentelemetry-python/pull/2947)) + +## [1.13.0-0.34b0](https://github.com/open-telemetry/opentelemetry-python/releases/tag/v1.13.0) - 2022-09-26 diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py b/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py index d7d825c9c5a..4fb9041d4c6 100644 --- a/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py +++ b/opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py @@ -227,7 +227,12 @@ def __init__( 100.0, 250.0, 500.0, + 750.0, 1000.0, + 2500.0, + 5000.0, + 7500.0, + 10000.0, ), record_min_max: bool = True, ): @@ -454,7 +459,12 @@ def __init__( 100.0, 250.0, 500.0, + 750.0, 1000.0, + 2500.0, + 5000.0, + 7500.0, + 10000.0, ), record_min_max: bool = True, ) -> None: diff --git a/opentelemetry-sdk/tests/metrics/test_aggregation.py b/opentelemetry-sdk/tests/metrics/test_aggregation.py index 5acf701b18c..9c9de1f2cbb 100644 --- a/opentelemetry-sdk/tests/metrics/test_aggregation.py +++ b/opentelemetry-sdk/tests/metrics/test_aggregation.py @@ -378,6 +378,28 @@ def test_collect(self): second_histogram.time_unix_nano, first_histogram.time_unix_nano ) + def test_boundaries(self): + self.assertEqual( + _ExplicitBucketHistogramAggregation(Mock(), 0)._boundaries, + ( + 0.0, + 5.0, + 10.0, + 25.0, + 50.0, + 75.0, + 100.0, + 250.0, + 500.0, + 750.0, + 1000.0, + 2500.0, + 5000.0, + 7500.0, + 10000.0, + ), + ) + class TestAggregationFactory(TestCase): def test_sum_factory(self):