Skip to content

Commit 21f32b5

Browse files
author
Jeel Mehta
committed
Revert "Release/v2.11.2 (aws-observability#1131)"
This reverts commit 4f5704f.
1 parent 4f5704f commit 21f32b5

File tree

68 files changed

+234
-6137
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

68 files changed

+234
-6137
lines changed

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

Lines changed: 82 additions & 695 deletions
Large diffs are not rendered by default.

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

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -245,19 +245,3 @@ jobs:
245245
with:
246246
aws-region: us-east-1
247247
caller-workflow-name: 'main-build'
248-
249-
# This validation is to ensure that all test workflows relevant to this repo are actually
250-
# being used in this repo, which is referring to all the other jobs in this file.
251-
#
252-
# If this starts failing, then it most likely means that new e2e test workflow was
253-
# added to `aws-observability/aws-application-signals-test-framework`, but was not
254-
# added to this file. It could also mean that a test in this file has been removed.
255-
#
256-
# If a particular test file is intended to not be tested in this repo and should not
257-
# be failing this particular validation, then choose one of the following options:
258-
# - Add the test file to the exclusions input (CSV format) to the workflow
259-
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml#L1)
260-
# - Update the `validate-e2e-tests-are-accounted-for` job to change which "workflow files are expected to be used by this repo"
261-
# (see: https://github.com/aws-observability/aws-application-signals-test-framework/blob/main/.github/workflows/validate-e2e-tests-are-accounted-for.yml)
262-
validate-all-tests-are-accounted-for:
263-
uses: aws-observability/aws-application-signals-test-framework/.github/workflows/validate-e2e-tests-are-accounted-for.yml@main

.github/workflows/e2e-tests-app-with-java-agent.yml

Lines changed: 15 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -167,18 +167,18 @@ jobs:
167167
VALIDATOR_COMMAND: -c spark-otel-trace-metric-validation.yml --endpoint http://app:4567 --metric-namespace aws-otel-integ-test -t ${{ github.run_id }}-${{ github.run_number }}
168168

169169
# publish status
170-
# publish-build-status:
171-
# needs: [ test_Spring_App_With_Java_Agent, test_Spark_App_With_Java_Agent, test_Spark_AWS_SDK_V1_App_With_Java_Agent ]
172-
# if: ${{ always() }}
173-
# uses: ./.github/workflows/publish-status.yml
174-
# with:
175-
# namespace: 'ADOT/GitHubActions'
176-
# repository: ${{ github.repository }}
177-
# branch: ${{ github.ref_name }}
178-
# workflow: ${{ inputs.caller-workflow-name }}
179-
# success: ${{ needs.test_Spring_App_With_Java_Agent.result == 'success' &&
180-
# needs.test_Spark_App_With_Java_Agent.result == 'success' &&
181-
# needs.test_Spark_AWS_SDK_V1_App_With_Java_Agent.result == 'success' }}
182-
# region: us-east-1
183-
# secrets:
184-
# roleArn: ${{ secrets.METRICS_ROLE_ARN }}
170+
publish-build-status:
171+
needs: [ test_Spring_App_With_Java_Agent, test_Spark_App_With_Java_Agent, test_Spark_AWS_SDK_V1_App_With_Java_Agent ]
172+
if: ${{ always() }}
173+
uses: ./.github/workflows/publish-status.yml
174+
with:
175+
namespace: 'ADOT/GitHubActions'
176+
repository: ${{ github.repository }}
177+
branch: ${{ github.ref_name }}
178+
workflow: ${{ inputs.caller-workflow-name }}
179+
success: ${{ needs.test_Spring_App_With_Java_Agent.result == 'success' &&
180+
needs.test_Spark_App_With_Java_Agent.result == 'success' &&
181+
needs.test_Spark_AWS_SDK_V1_App_With_Java_Agent.result == 'success' }}
182+
region: us-east-1
183+
secrets:
184+
roleArn: ${{ secrets.METRICS_ROLE_ARN }}

.github/workflows/main-build.yml

Lines changed: 13 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -263,21 +263,16 @@ jobs:
263263
adot-image-name: ${{ needs.build.outputs.staging-image }}
264264

265265
publish-build-status:
266-
name: "Publish Main Build Status"
267-
needs: [ build, e2e-test, contract-tests, application-signals-lambda-layer-build, application-signals-e2e-test ]
268-
runs-on: ubuntu-latest
269-
if: always()
270-
steps:
271-
- name: Configure AWS Credentials for emitting metrics
272-
uses: aws-actions/configure-aws-credentials@v4
273-
with:
274-
role-to-assume: ${{ secrets.METRICS_ROLE_ARN }}
275-
aws-region: us-east-1
276-
277-
- name: Publish main build status
278-
run: |
279-
value="${{ needs.build.result == 'success' && needs.e2e-test.result == 'success' && needs.contract-tests.result == 'success' && needs.application-signals-lambda-layer-build.result == 'success' && needs.application-signals-e2e-test.result == 'success' && '0.0' || '1.0' }}"
280-
aws cloudwatch put-metric-data --namespace 'ADOT/GitHubActions' \
281-
--metric-name Failure \
282-
--dimensions repository=${{ github.repository }},branch=${{ github.ref_name }},workflow=main_build \
283-
--value $value
266+
needs: [ build, contract-tests ]
267+
if: ${{ always() }}
268+
uses: ./.github/workflows/publish-status.yml
269+
with:
270+
namespace: 'ADOT/GitHubActions'
271+
repository: ${{ github.repository }}
272+
branch: ${{ github.ref_name }}
273+
workflow: main-build
274+
success: ${{ needs.build.result == 'success' &&
275+
needs.contract-tests.result == 'success' }}
276+
region: us-east-1
277+
secrets:
278+
roleArn: ${{ secrets.METRICS_ROLE_ARN }}

.github/workflows/owasp.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,15 +97,15 @@ jobs:
9797
id: high_scan_v2
9898
uses: ./.github/actions/image_scan
9999
with:
100-
image-ref: "public.ecr.aws/aws-observability/adot-autoinstrumentation-java:v2.11.1"
100+
image-ref: "public.ecr.aws/aws-observability/adot-autoinstrumentation-java:v2.11.0"
101101
severity: 'CRITICAL,HIGH'
102102

103103
- name: Perform low image scan on v2
104104
if: always()
105105
id: low_scan_v2
106106
uses: ./.github/actions/image_scan
107107
with:
108-
image-ref: "public.ecr.aws/aws-observability/adot-autoinstrumentation-java:v2.11.1"
108+
image-ref: "public.ecr.aws/aws-observability/adot-autoinstrumentation-java:v2.11.0"
109109
severity: 'MEDIUM,LOW,UNKNOWN'
110110

111111
- name: Configure AWS Credentials for emitting metrics

.github/workflows/release-build.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ permissions:
2121

2222
jobs:
2323
build:
24-
environment: Release
2524
runs-on: ubuntu-latest
2625
steps:
2726
- uses: actions/checkout@v4

.github/workflows/release-lambda.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ on:
99
aws_region:
1010
description: 'Deploy to aws regions'
1111
required: true
12-
default: 'us-east-1, us-east-2, us-west-1, us-west-2, ap-south-1, ap-northeast-3, ap-northeast-2, ap-southeast-1, ap-southeast-2, ap-northeast-1, ca-central-1, eu-central-1, eu-west-1, eu-west-2, eu-west-3, eu-north-1, sa-east-1, af-south-1, ap-east-1, ap-south-2, ap-southeast-3, ap-southeast-4, eu-central-2, eu-south-1, eu-south-2, il-central-1, me-central-1, me-south-1, ap-southeast-5, ap-southeast-7, mx-central-1, ca-west-1, cn-north-1, cn-northwest-1'
12+
default: 'us-east-1, us-east-2, us-west-1, us-west-2, ap-south-1, ap-northeast-3, ap-northeast-2, ap-southeast-1, ap-southeast-2, ap-northeast-1, ca-central-1, eu-central-1, eu-west-1, eu-west-2, eu-west-3, eu-north-1, sa-east-1, af-south-1, ap-east-1, ap-south-2, ap-southeast-3, ap-southeast-4, eu-central-2, eu-south-1, eu-south-2, il-central-1, me-central-1, me-south-1'
1313

1414
env:
15-
COMMERCIAL_REGIONS: us-east-1, us-east-2, us-west-1, us-west-2, ap-south-1, ap-northeast-3, ap-northeast-2, ap-southeast-1, ap-southeast-2, ap-northeast-1, ca-central-1, eu-central-1, eu-west-1, eu-west-2, eu-west-3, eu-north-1, sa-east-1, ap-southeast-5, ap-southeast-7, mx-central-1, ca-west-1, cn-north-1, cn-northwest-1
15+
COMMERCIAL_REGIONS: us-east-1, us-east-2, us-west-1, us-west-2, ap-south-1, ap-northeast-3, ap-northeast-2, ap-southeast-1, ap-southeast-2, ap-northeast-1, ca-central-1, eu-central-1, eu-west-1, eu-west-2, eu-west-3, eu-north-1, sa-east-1
1616
LAYER_NAME: AWSOpenTelemetryDistroJava
1717

1818
permissions:
@@ -21,7 +21,6 @@ permissions:
2121

2222
jobs:
2323
build-layer:
24-
environment: Release
2524
runs-on: ubuntu-latest
2625
outputs:
2726
aws_regions_json: ${{ steps.set-matrix.outputs.aws_regions_json }}

.github/workflows/release-udp-exporter.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ jobs:
2121
id-token: write
2222

2323
release-udp-exporter:
24-
environment: Release
2524
runs-on: ubuntu-latest
2625
needs: validate-udp-exporter-e2e-test
2726
steps:

0 commit comments

Comments
 (0)