Skip to content

Commit

Permalink
Fix OTLP exporter artifact name in an error message (#5541)
Browse files Browse the repository at this point in the history
  • Loading branch information
Mateusz Rzeszutek committed Jun 15, 2023
1 parent 0530e63 commit 26dd5d8
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ class LogRecordExporterConfiguration {
EXPORTER_ARTIFACT_ID_BY_NAME = new HashMap<>();
EXPORTER_ARTIFACT_ID_BY_NAME.put("logging", "opentelemetry-exporter-logging");
EXPORTER_ARTIFACT_ID_BY_NAME.put("logging-otlp", "opentelemetry-exporter-logging-otlp");
EXPORTER_ARTIFACT_ID_BY_NAME.put("otlp", "opentelemetry-exporter-otlp-logs");
EXPORTER_ARTIFACT_ID_BY_NAME.put("otlp", "opentelemetry-exporter-otlp");
}

// Visible for test
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ void configureExporter_KnownSpiExportersNotOnClasspath() {
assertThatThrownBy(() -> configureExporter("otlp", spiExportersManager))
.isInstanceOf(ConfigurationException.class)
.hasMessage(
"otel.logs.exporter set to \"otlp\" but opentelemetry-exporter-otlp-logs"
"otel.logs.exporter set to \"otlp\" but opentelemetry-exporter-otlp"
+ " not found on classpath. Make sure to add it as a dependency.");

// Unrecognized exporter
Expand Down

0 comments on commit 26dd5d8

Please sign in to comment.