Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit 6ebd1e5

Browse files
author
Andrew Xue
committed
address comments
1 parent 2821882 commit 6ebd1e5

File tree

1 file changed

+21
-4
lines changed
  • ext/opentelemetry-exporter-cloud-monitoring/src/opentelemetry/exporter/cloud_monitoring

1 file changed

+21
-4
lines changed

ext/opentelemetry-exporter-cloud-monitoring/src/opentelemetry/exporter/cloud_monitoring/__init__.py

Lines changed: 21 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -108,9 +108,19 @@ def _get_metric_descriptor(
108108
descriptor["value_type"] = MetricDescriptor.ValueType.INT64
109109
elif record.metric.value_type == float:
110110
descriptor["value_type"] = MetricDescriptor.ValueType.DOUBLE
111-
descriptor = self.client.create_metric_descriptor(
112-
self.project_name, MetricDescriptor(**descriptor)
113-
)
111+
proto_descriptor = MetricDescriptor(**descriptor)
112+
try:
113+
descriptor = self.client.create_metric_descriptor(
114+
self.project_name, proto_descriptor
115+
)
116+
# pylint: disable=broad-except
117+
except Exception as ex:
118+
logger.error(
119+
"Failed to create metric descriptor %s",
120+
proto_descriptor,
121+
exc_info=ex,
122+
)
123+
return None
114124
self._metric_descriptors[descriptor_type] = descriptor
115125
return descriptor
116126

@@ -147,5 +157,12 @@ def export(
147157
point.interval.end_time.seconds = int(seconds)
148158
point.interval.end_time.nanos = int(nanos)
149159
all_series.append(series)
150-
self._batch_write(all_series)
160+
try:
161+
self._batch_write(all_series)
162+
# pylint: disable=broad-except
163+
except Exception as ex:
164+
logger.error(
165+
"Error while writing to Cloud Monitoring", exc_info=ex
166+
)
167+
return MetricsExportResult.FAILURE
151168
return MetricsExportResult.SUCCESS

0 commit comments

Comments
 (0)