Skip to content

Conversation

@jaydeluca
Copy link
Member

I was reading the code in these modules and noticed they didn't follow the standard pattern of using a Singletons class for the instrumenters, so I renamed the helper classes to follow that pattern. Also cleaned up some test code visibility.

@jaydeluca jaydeluca requested a review from a team as a code owner August 4, 2025 18:25
@trask trask merged commit b40f923 into open-telemetry:main Aug 4, 2025
89 checks passed
lukeina2z added a commit to aws-observability/aws-otel-java-instrumentation that referenced this pull request Oct 27, 2025
This PR updates the upstream dependency OpenTelemetry Java Agent to
their 2025 Sep. release version 2.20.1
(opentelemetry-instrumentation:2.20.1).

By doing so, the following upstream OpenTelemetry dependencies are
automatically updated along with the Java Agent: opentelemetry-api:
1.54.1
opentelemetry-sdk: 1.54.1
opentelemetry-semconv: 1.37.0

Conflicts between the ADOT patch and the two upstream PRs below are
resolved in this PR.

Rename lambda instrumentation helper class

open-telemetry/opentelemetry-java-instrumentation#14371

Let AWS Lambda SQS handlers report partial batch failures

open-telemetry/opentelemetry-java-instrumentation#14468

Tests Performed
Unit tests: ./gradlew build test
Smoke/contract tests: ./gradlew
appsignals-tests:contract-tests:contractTests

Manual E2E tests:
Spring Boot EC2 sample app
Java Lambda function
Spring Boot web application hosted on AWS Lambda

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
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