Skip to content

Commit

Permalink
Remove opentelemetry-exporter-logging because it doesnt work on RDE i…
Browse files Browse the repository at this point in the history
…f you have multiple ServiceLoader Producers of the same interface
  • Loading branch information
royteeuwen committed Oct 23, 2024
1 parent 1776ab5 commit 0902750
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 13 deletions.
3 changes: 2 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ OTEL_EXPORTER_OTLP_ENDPOINT=http://otel-collector:4318
OTEL_SERVICE_NAME=aem-author
```

For local debugging you can use the logging exporter to print to the console.
For local debugging, add the `opentelemetry-exporter-logging` bundle to your `all` package in start level `15` with a specific profile for
local development, so that you can use the logging exporter to print to the console.

```text
OTEL_TRACES_EXPORTER=logging
Expand Down
12 changes: 0 additions & 12 deletions opentelemetry-okhttp-exporter/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -57,13 +57,6 @@
/apps/aemaacs-opentelemetry-instrumentation-opentelemetry-okhttp-exporter-packages/application/install/15
</target>
</embedded>
<embedded>
<groupId>be.orbinson.osgi</groupId>
<artifactId>opentelemetry-exporter-logging</artifactId>
<target>
/apps/aemaacs-opentelemetry-instrumentation-opentelemetry-okhttp-exporter-packages/application/install/15
</target>
</embedded>
<embedded>
<groupId>be.orbinson.osgi</groupId>
<artifactId>opentelemetry-exporter-otlp</artifactId>
Expand Down Expand Up @@ -366,11 +359,6 @@
<artifactId>opentelemetry-exporter-common</artifactId>
<version>${opentelemetry.version}.0001</version>
</dependency>
<dependency>
<groupId>be.orbinson.osgi</groupId>
<artifactId>opentelemetry-exporter-logging</artifactId>
<version>${opentelemetry.version}.0001</version>
</dependency>
<dependency>
<groupId>be.orbinson.osgi</groupId>
<artifactId>opentelemetry-exporter-otlp</artifactId>
Expand Down

0 comments on commit 0902750

Please sign in to comment.