Skip to content

Commit b03319c

Browse files
[currency] fix metrics export endpoint (open-telemetry#904)
* export metrics endpoint to default config Signed-off-by: Pierre Tessier <[email protected]> * export metrics endpoint to default config Signed-off-by: Pierre Tessier <[email protected]> * export metrics endpoint to default config Signed-off-by: Pierre Tessier <[email protected]> --------- Signed-off-by: Pierre Tessier <[email protected]> Co-authored-by: Carter Socha <[email protected]>
1 parent 210649a commit b03319c

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

CHANGELOG.md

+2
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ release.
2929
([#880](https://github.com/open-telemetry/opentelemetry-demo/pull/880))
3030
* [currencyservice] Fix OTel C++ build and update OTel version to 1.9.0
3131
([#886](https://github.com/open-telemetry/opentelemetry-demo/pull/886))
32+
* [currencyservice] Fix OTLP export to use default env vars
33+
([#904](https://github.com/open-telemetry/opentelemetry-demo/pull/904))
3234
* [featureflagservice] Bump OTP version to 26.0
3335
([#903](https://github.com/open-telemetry/opentelemetry-demo/pull/903))
3436

src/currencyservice/src/meter_common.h

-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ namespace
2626
otlp_exporter::OtlpGrpcMetricExporterOptions otlpOptions;
2727

2828
// Configuration via environment variable not supported yet
29-
otlpOptions.endpoint = "otelcol:4317";
3029
otlpOptions.aggregation_temporality = metric_sdk::AggregationTemporality::kDelta;
3130
auto exporter = otlp_exporter::OtlpGrpcMetricExporterFactory::Create(otlpOptions);
3231

0 commit comments

Comments
 (0)