Skip to content

Commit f83cb9d

Browse files
authored
Add ECS E2E test into main build (#890)
This PR do the following two things: 1. Fix [recent main build failure](https://github.com/aws-observability/aws-otel-java-instrumentation/actions/runs/10926150649/job/30330248240) by updating Dockerfile syntax. 2. Add ECS E2E test entry point into main build. An example of succeed test workflow: https://github.com/aws-observability/aws-otel-java-instrumentation/actions/runs/10926900740 (E2E operator test failure is expected, will plan on the effort fixing it). By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 4afa55e commit f83cb9d

File tree

2 files changed

+9
-1
lines changed

2 files changed

+9
-1
lines changed

.github/workflows/application-signals-e2e-test.yml

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -76,6 +76,14 @@ jobs:
7676
java-k8s-e2e-test:
7777
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/java-k8s-test.yml@main
7878
secrets: inherit
79+
with:
80+
aws-region: us-east-1
81+
adot-image-name: ${{ inputs.adot-image-name }}
82+
caller-workflow-name: 'main-build'
83+
84+
java-ecs-e2e-test:
85+
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/java-ecs-test.yml@main
86+
secrets: inherit
7987
with:
8088
aws-region: us-east-1
8189
adot-image-name: ${{ inputs.adot-image-name }}

Dockerfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,4 +51,4 @@ ARG ADOT_JAVA_VERSION
5151
COPY --from=builder /usr/src/cp-utility/bin/cp-utility /bin/cp
5252

5353

54-
COPY --chmod=go+r ./otelagent/build/libs/aws-opentelemetry-agent-${ADOT_JAVA_VERSION}.jar /javaagent.jar
54+
COPY --chmod=0644 ./otelagent/build/libs/aws-opentelemetry-agent-${ADOT_JAVA_VERSION}.jar /javaagent.jar

0 commit comments

Comments
 (0)