[currency] fix metrics export endpoint (open-telemetry#904)

* export metrics endpoint to default config Signed-off-by: Pierre Tessier <pierre@pierretessier.com> * export metrics endpoint to default config Signed-off-by: Pierre Tessier <pierre@pierretessier.com> * export metrics endpoint to default config Signed-off-by: Pierre Tessier <pierre@pierretessier.com> --------- Signed-off-by: Pierre Tessier <pierre@pierretessier.com> Co-authored-by: Carter Socha <43380952+cartersocha@users.noreply.github.com>
AlexanderWert · Jul 24, 2023 · 3b89c9a · 3b89c9a
1 parent 93ef675
commit 3b89c9a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,6 +29,8 @@ release.
   ([#880](https://github.com/open-telemetry/opentelemetry-demo/pull/880))
 * [currencyservice] Fix OTel C++ build and update OTel version to 1.9.0
   ([#886](https://github.com/open-telemetry/opentelemetry-demo/pull/886))
+* [currencyservice] Fix OTLP export to use default env vars
+  ([#904](https://github.com/open-telemetry/opentelemetry-demo/pull/904))
 * [featureflagservice] Bump OTP version to 26.0
   ([#903](https://github.com/open-telemetry/opentelemetry-demo/pull/903))
 

diff --git a/src/currencyservice/src/meter_common.h b/src/currencyservice/src/meter_common.h
@@ -26,7 +26,6 @@ namespace
     otlp_exporter::OtlpGrpcMetricExporterOptions otlpOptions;
 
     // Configuration via environment variable not supported yet
-    otlpOptions.endpoint = "otelcol:4317";
     otlpOptions.aggregation_temporality = metric_sdk::AggregationTemporality::kDelta;
     auto exporter = otlp_exporter::OtlpGrpcMetricExporterFactory::Create(otlpOptions);