Adding Resource to MetricRecord (#1209)
Co-authored-by: Diego Hurtado <ocelotl@users.noreply.github.com>
This commit is contained in:
parent
6e3965be16
commit
56ffabdddf
|
|
@ -67,7 +67,10 @@ class TestPrometheusMetricExporter(unittest.TestCase):
|
|||
def test_export(self):
|
||||
with self._registry_register_patch:
|
||||
record = MetricRecord(
|
||||
self._test_metric, self._labels_key, SumAggregator(),
|
||||
self._test_metric,
|
||||
self._labels_key,
|
||||
SumAggregator(),
|
||||
get_meter_provider().resource,
|
||||
)
|
||||
exporter = PrometheusMetricsExporter()
|
||||
result = exporter.export([record])
|
||||
|
|
@ -86,7 +89,9 @@ class TestPrometheusMetricExporter(unittest.TestCase):
|
|||
aggregator.update(123)
|
||||
aggregator.update(456)
|
||||
aggregator.take_checkpoint()
|
||||
record = MetricRecord(metric, key_labels, aggregator)
|
||||
record = MetricRecord(
|
||||
metric, key_labels, aggregator, get_meter_provider().resource
|
||||
)
|
||||
collector = CustomCollector("testprefix")
|
||||
collector.add_metrics_data([record])
|
||||
result_bytes = generate_latest(collector)
|
||||
|
|
@ -104,7 +109,9 @@ class TestPrometheusMetricExporter(unittest.TestCase):
|
|||
aggregator = SumAggregator()
|
||||
aggregator.update(123)
|
||||
aggregator.take_checkpoint()
|
||||
record = MetricRecord(metric, key_labels, aggregator)
|
||||
record = MetricRecord(
|
||||
metric, key_labels, aggregator, get_meter_provider().resource
|
||||
)
|
||||
collector = CustomCollector("testprefix")
|
||||
collector.add_metrics_data([record])
|
||||
|
||||
|
|
@ -132,7 +139,9 @@ class TestPrometheusMetricExporter(unittest.TestCase):
|
|||
)
|
||||
labels = {"environment": "staging"}
|
||||
key_labels = get_dict_as_key(labels)
|
||||
record = MetricRecord(metric, key_labels, None)
|
||||
record = MetricRecord(
|
||||
metric, key_labels, None, get_meter_provider().resource
|
||||
)
|
||||
collector = CustomCollector("testprefix")
|
||||
collector.add_metrics_data([record])
|
||||
collector.collect()
|
||||
|
|
|
|||
Loading…
Reference in New Issue