Skip to content

Fix duplicate class error on Android build - #5882

Merged
trask merged 1 commit into
open-telemetry:mainfrom
mateuszrzeszutek:fix-duplicate-class
Apr 19, 2022
Merged

Fix duplicate class error on Android build#5882
trask merged 1 commit into
open-telemetry:mainfrom
mateuszrzeszutek:fix-duplicate-class

Conversation

@mateuszrzeszutek

Copy link
Copy Markdown
Member

I tried to bump the OTel instrumentation versions to 1.13 in our Splunk Android instrumentation repo (signalfx/splunk-otel-android#276), got this error:

Duplicate class io.opentelemetry.instrumentation.api.package-info found in modules opentelemetry-instrumentation-api-1.13.0-alpha (io.opentelemetry.instrumentation:opentelemetry-instrumentation-api:1.13.0-alpha) and opentelemetry-instrumentation-api-semconv-1.13.0-alpha (io.opentelemetry.instrumentation:opentelemetry-instrumentation-api-semconv:1.13.0-alpha)

Looks like that file only appears once in the javaagent, and it's probably only a problem for the Android gradle plugin.
@open-telemetry/java-instrumentation-maintainers do you think this warrants a patch release?

@mateuszrzeszutek
mateuszrzeszutek requested a review from a team April 19, 2022 10:51

@trask trask left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://github.com/orgs/open-telemetry/teams/java-instrumentation-maintainers do you think this warrants a patch release?

I think so. I need to test out the patch release workflow anyways.

@trask
trask merged commit 492c022 into open-telemetry:main Apr 19, 2022
github-actions Bot pushed a commit that referenced this pull request Apr 19, 2022
trask pushed a commit that referenced this pull request Apr 19, 2022
Co-authored-by: Mateusz Rzeszutek <mrzeszutek@splunk.com>
@mateuszrzeszutek
mateuszrzeszutek deleted the fix-duplicate-class branch April 20, 2022 06:21
RashmiRam pushed a commit to RashmiRam/opentelemetry-auto-instr-java that referenced this pull request May 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants