Skip to content

Commit dbc32a1

Browse files
committed
reverting to using adot1 after syncing with Michael He
1 parent 1a71fb4 commit dbc32a1

File tree

3 files changed

+10
-18
lines changed

3 files changed

+10
-18
lines changed

.github/patches/opentelemetry-java-instrumentation.patch

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3177,8 +3177,8 @@ index fdf57bdbea..c38a2e00f3 100644
31773177
@@ -1,5 +1,5 @@
31783178
-val stableVersion = "1.32.1"
31793179
-val alphaVersion = "1.32.1-alpha"
3180-
+val stableVersion = "1.32.1-adot2"
3181-
+val alphaVersion = "1.32.1-adot2-alpha"
3180+
+val stableVersion = "1.32.1-adot1"
3181+
+val alphaVersion = "1.32.1-adot1-alpha"
31823182

31833183
allprojects {
31843184
if (findProperty("otel.stable") != "true") {

lambda-layer/build-layer.sh

100644100755
Lines changed: 7 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,12 @@ ADOT_VERSION="1.32.1"
1212
git clone https://github.com/open-telemetry/opentelemetry-java-instrumentation.git
1313
pushd opentelemetry-java-instrumentation
1414
git checkout v${OTEL_VERSION} -b tag-v${OTEL_VERSION}
15+
16+
# This patch is for Lambda related context propagation
1517
patch -p1 < "$SOURCEDIR"/patches/opentelemetry-java-instrumentation.patch
18+
# There is another patch in the .github/patches directory for other changes. We should apply them too for consistency.
19+
patch -p1 < "$SOURCEDIR"/../.github/patches/opentelemetry-java-instrumentation.patch
20+
1621
git add -A
1722
git commit -m "Create patch version"
1823
./gradlew publishToMavenLocal
@@ -26,22 +31,9 @@ patch -p1 < "${SOURCEDIR}"/patches/aws-otel-java-instrumentation.patch
2631
CI=false ./gradlew publishToMavenLocal -Prelease.version=${ADOT_VERSION}-adot-lambda1
2732
popd
2833

29-
# Build ADOT Lambda Java SDK Layer Code
34+
## Build ADOT Lambda Java SDK Layer Code
3035
./gradlew build
3136

32-
## Move the ADOT Lambda Java SDK code into OTel Lambda Java folder - TODO: probably not needed since this is related to wrapper
33-
#mkdir -p ../opentelemetry-lambda/java/layer-wrapper/build/extensions
34-
#cp ./build/libs/aws-otel-lambda-java-extensions.jar ../opentelemetry-lambda/java/layer-wrapper/build/extensions
35-
36-
## Combine Java Agent build and ADOT Collector
37-
#pushd ./layer-javaagent/build/distributions || exit
38-
#unzip -qo opentelemetry-javaagent-layer.zip
39-
#rm opentelemetry-javaagent-layer.zip
40-
#mv otel-handler otel-handler-upstream
41-
#cp "$SOURCEDIR"/scripts/otel-handler .
42-
43-
# Copy ADOT Java Agent downloaded using Gradle task
37+
## Copy ADOT Java Agent downloaded using Gradle task and bundle it with the Lambda handler script
4438
cp "$SOURCEDIR"/build/javaagent/aws-opentelemetry-agent*.jar ./opentelemetry-javaagent.jar
45-
#unzip -qo ../../../../collector/build/opentelemetry-collector-layer-$1.zip - TODO: bundling collector not needed
4639
zip -qr opentelemetry-javaagent-layer.zip opentelemetry-javaagent.jar otel-handler
47-
#popd || exit

lambda-layer/patches/aws-otel-java-instrumentation.patch

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ index 9493189..6090207 100644
66
val TEST_SNAPSHOTS = rootProject.findProperty("testUpstreamSnapshots") == "true"
77

88
// This is the version of the upstream instrumentation BOM
9-
-val otelVersion = "1.32.1-adot2"
9+
-val otelVersion = "1.32.1-adot1"
1010
+val otelVersion = "1.32.1-adot-lambda1"
1111
val otelSnapshotVersion = "1.33.0"
1212
val otelAlphaVersion = if (!TEST_SNAPSHOTS) "$otelVersion-alpha" else "$otelSnapshotVersion-alpha-SNAPSHOT"

0 commit comments

Comments
 (0)