Skip to content

Commit 3284c00

Browse files
committed
Make advisory attribute private
1 parent dd5adcf commit 3284c00

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/aggregation.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1277,8 +1277,8 @@ def _create_aggregation(
12771277

12781278
if isinstance(instrument, Histogram):
12791279
boundaries: Optional[Sequence[float]] = (
1280-
instrument.advisory.get("explicit_bucket_boundaries")
1281-
if instrument.advisory is not None
1280+
instrument._advisory.get("explicit_bucket_boundaries")
1281+
if instrument._advisory is not None
12821282
else None
12831283
)
12841284
return _ExplicitBucketHistogramAggregation(

opentelemetry-sdk/src/opentelemetry/sdk/metrics/_internal/instrument.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ def __init__(
236236
instrumentation_scope=instrumentation_scope,
237237
measurement_consumer=measurement_consumer,
238238
)
239-
self.advisory = advisory
239+
self._advisory = advisory
240240

241241
def __new__(cls, *args, **kwargs):
242242
if cls is Histogram:

opentelemetry-sdk/tests/metrics/test_metrics.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -511,7 +511,7 @@ def test_create_histogram_with_advisory(self):
511511
self.assertIsInstance(histogram, Histogram)
512512
self.assertEqual(histogram.name, "name")
513513
self.assertEqual(
514-
histogram.advisory, {"explicit_bucket_boundaries": [0, 1, 2]}
514+
histogram._advisory, {"explicit_bucket_boundaries": [0, 1, 2]}
515515
)
516516

517517
def test_create_histogram_advisory_validation(self):

0 commit comments

Comments
 (0)