Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Commit cbe26fd

Browse files
authored
Merge pull request #93 from lzchen/hot-fix
2 parents 66321f4 + f1fc577 commit cbe26fd

File tree

3 files changed

+15
-4
lines changed

3 files changed

+15
-4
lines changed

azure_monitor/CHANGELOG.md

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,12 @@
22

33
## Unreleased
44

5+
## 0.3b.1
6+
Released 2020-05-21
7+
8+
- Fix metrics exporter serialization bug
9+
([#92](https://github.com/microsoft/opentelemetry-azure-monitor-python/pull/92))
10+
511
## 0.3b.0
612
Released 2020-05-19
713

azure_monitor/src/azure_monitor/export/metrics/__init__.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,12 @@ def export(
3535
self, metric_records: Sequence[MetricRecord]
3636
) -> MetricsExportResult:
3737
envelopes = list(map(self._metric_to_envelope, metric_records))
38-
envelopes = self._apply_telemetry_processors(envelopes)
38+
envelopes = list(
39+
map(
40+
lambda x: x.to_dict(),
41+
self._apply_telemetry_processors(envelopes),
42+
)
43+
)
3944
try:
4045
result = self._transmit(envelopes)
4146
if result == ExportResult.FAILED_RETRYABLE:

azure_monitor/tests/metrics/test_metrics.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ def test_export(self, mte, transmit):
107107
CounterAggregator(), self._test_labels, self._test_metric
108108
)
109109
exporter = self._exporter
110-
mte.return_value = []
110+
mte.return_value = Envelope()
111111
transmit.return_value = ExportResult.SUCCESS
112112
result = exporter.export([record])
113113
self.assertEqual(result, MetricsExportResult.SUCCESS)
@@ -124,7 +124,7 @@ def test_export_failed_retryable(self, mte, transmit):
124124
)
125125
exporter = self._exporter
126126
transmit.return_value = ExportResult.FAILED_RETRYABLE
127-
mte.return_value = []
127+
mte.return_value = Envelope()
128128
storage_mock = mock.Mock()
129129
exporter.storage.put = storage_mock
130130
result = exporter.export([record])
@@ -143,7 +143,7 @@ def test_export_exception(self, mte, transmit, logger_mock):
143143
CounterAggregator(), self._test_labels, self._test_metric
144144
)
145145
exporter = self._exporter
146-
mte.return_value = []
146+
mte.return_value = Envelope()
147147
transmit.side_effect = throw(Exception)
148148
result = exporter.export([record])
149149
self.assertEqual(result, MetricsExportResult.FAILURE)

0 commit comments

Comments
 (0)