Skip to content

Commit 79c860d

Browse files
committed
Test validation adjustments, workflow adjustments
update emf dimensions, add resourcedetection processor switch to otlp endpoint update collector config tmp use jj22ee script for ocb use generic platform, update test validations to remove some expected metadata expected UnmappedOperation, maybe expect no RemoteEnvironment, RemoteService uses IP --remote-service-ip addition client-call has only 1 EMF entry update metrics with UnmappedOperation query RemoteServiceIp remove local InternalOperation in client-call change region to us-east-1 change otel col test cluster name fix update validations for traces update metadata in entity trace touch-ups workflow touch ups undo hardcoded region cron job
1 parent 3cc5c2e commit 79c860d

19 files changed

+295
-810
lines changed

.github/workflows/java-eks-otlp-ocb-canary.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,12 +9,12 @@
99
name: Java EKS OTLP/OCB Enablement Canary Test
1010
on:
1111
# TODO: Reintroduce cron-job
12-
# schedule:
13-
# - cron: '12,37 * * * *' # run the workflow at 12th and 37th minute of every hour
12+
schedule:
13+
- cron: '12,37 * * * *' # run the workflow at 12th and 37th minute of every hour
1414
workflow_dispatch: # be able to run the workflow on demand
15-
push:
16-
branches:
17-
- otlp-ocb
15+
# push:
16+
# branches:
17+
# - otlp-ocb
1818

1919
permissions:
2020
id-token: write

.github/workflows/java-eks-otlp-ocb-test.yml

Lines changed: 17 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -30,9 +30,9 @@ on:
3030
type: string
3131
outputs:
3232
job-started:
33-
value: ${{ jobs.metric-limiter.outputs.job-started }}
33+
value: ${{ jobs.java-eks-otlp-ocb.outputs.job-started }}
3434
validation-result:
35-
value: ${{ jobs.metric-limiter.outputs.validation-result }}
35+
value: ${{ jobs.java-eks-otlp-ocb.outputs.validation-result }}
3636

3737
permissions:
3838
id-token: write
@@ -54,7 +54,7 @@ env:
5454
TEST_RESOURCES_FOLDER: ${GITHUB_WORKSPACE}
5555

5656
jobs:
57-
test:
57+
java-eks-otlp-ocb:
5858
runs-on: ubuntu-latest
5959
timeout-minutes: 30
6060
outputs:
@@ -94,7 +94,7 @@ jobs:
9494
uses: ./.github/workflows/actions/execute_and_retry
9595
with:
9696
pre-command: "mkdir enablement-script && cd enablement-script"
97-
command: "wget https://raw.githubusercontent.com/aws-observability/application-signals-demo/refs/heads/ocb/scripts/eks/appsignals/enable-app-signals-ocb.sh"
97+
command: "wget https://raw.githubusercontent.com/jj22ee/application-signals-demo/refs/heads/ocb/scripts/eks/appsignals/enable-app-signals-ocb.sh"
9898
cleanup: "rm -f enable-app-signals-ocb.sh"
9999
post-command: "chmod +x enable-app-signals-ocb.sh"
100100

@@ -202,7 +202,9 @@ jobs:
202202
- name: Install OTel Collector
203203
uses: ./.github/workflows/actions/execute_and_retry
204204
with:
205-
command: "cd ${{ env.TEST_RESOURCES_FOLDER }}/terraform/java/eks-otlp-ocb/util && kubectl apply -f ./appsignals-collector.yaml -n ${{ env.SAMPLE_APP_NAMESPACE }}"
205+
command: "cd ${{ env.TEST_RESOURCES_FOLDER }}/terraform/java/eks-otlp-ocb/util && \
206+
sed -i -e 's/${E2E_TEST_AWS_REGION}/${{ env.E2E_TEST_AWS_REGION }}/g' ./appsignals-collector.yaml && \
207+
kubectl apply -f ./appsignals-collector.yaml -n ${{ env.SAMPLE_APP_NAMESPACE }}"
206208
cleanup: "kubectl delete -f ./appsignals-collector.yaml -n ${{ env.SAMPLE_APP_NAMESPACE }} && \
207209
aws eks update-kubeconfig --name ${{ env.CLUSTER_NAME }} --region ${{ env.E2E_TEST_AWS_REGION }}"
208210
max_retry: 3
@@ -271,60 +273,54 @@ jobs:
271273
run: ./gradlew validator:run --args='-c java/eks-otlp-ocb/log-validation.yml
272274
--testing-id ${{ env.TESTING_ID }}
273275
--endpoint http://${{ env.APP_ENDPOINT }}
274-
--region us-west-2
276+
--region ${{ env.E2E_TEST_AWS_REGION }}
275277
--account-id ${{ env.ACCOUNT_ID }}
276278
--metric-namespace ${{ env.METRIC_NAMESPACE }}
277279
--log-group ${{ env.LOG_GROUP_NAME }}
278280
--app-namespace ${{ env.SAMPLE_APP_NAMESPACE }}
279-
--platform-info appsignals-ocb-demo
281+
--platform-info ${{ env.CLUSTER_NAME }}
280282
--service-name sample-application-${{ env.TESTING_ID }}
281283
--remote-service-deployment-name ${{ env.REMOTE_SERVICE_DEPLOYMENT_NAME }}
284+
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
282285
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
283286
--rollup'
284-
# TODO: Undo hardcode
285-
# --platform-info ${{ env.CLUSTER_NAME }}
286-
# --region ${{ env.E2E_TEST_AWS_REGION }}
287287

288288
- name: Call endpoints and validate generated metrics
289289
id: metric-validation
290290
if: (steps.deploy-sample-app.outcome == 'success' || steps.log-validation.outcome == 'failure') && !cancelled()
291291
run: ./gradlew validator:run --args='-c java/eks-otlp-ocb/metric-validation.yml
292292
--testing-id ${{ env.TESTING_ID }}
293293
--endpoint http://${{ env.APP_ENDPOINT }}
294-
--region us-west-2
294+
--region ${{ env.E2E_TEST_AWS_REGION }}
295295
--account-id ${{ env.ACCOUNT_ID }}
296296
--metric-namespace ${{ env.METRIC_NAMESPACE }}
297297
--log-group ${{ env.LOG_GROUP_NAME }}
298298
--app-namespace ${{ env.SAMPLE_APP_NAMESPACE }}
299-
--platform-info appsignals-ocb-demo
299+
--platform-info ${{ env.CLUSTER_NAME }}
300300
--service-name sample-application-${{ env.TESTING_ID }}
301301
--remote-service-name sample-remote-application-${{ env.TESTING_ID }}
302302
--remote-service-deployment-name ${{ env.REMOTE_SERVICE_DEPLOYMENT_NAME }}
303+
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
303304
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
304305
--rollup'
305-
# TODO: Undo hardcodes:
306-
# --platform-info ${{ env.CLUSTER_NAME }}
307-
# --region ${{ env.E2E_TEST_AWS_REGION }}
308306

309307
- name: Call endpoints and validate generated traces
310308
id: trace-validation
311309
if: (steps.deploy-sample-app.outcome == 'success' || steps.log-validation.outcome == 'failure' || steps.metric-validation.outcome == 'failure') && !cancelled()
312310
run: ./gradlew validator:run --args='-c java/eks-otlp-ocb/trace-validation.yml
313311
--testing-id ${{ env.TESTING_ID }}
314312
--endpoint http://${{ env.APP_ENDPOINT }}
315-
--region us-west-2
313+
--region ${{ env.E2E_TEST_AWS_REGION }}
316314
--account-id ${{ env.ACCOUNT_ID }}
317315
--metric-namespace ${{ env.METRIC_NAMESPACE }}
318316
--log-group ${{ env.LOG_GROUP_NAME }}
319317
--app-namespace ${{ env.SAMPLE_APP_NAMESPACE }}
320-
--platform-info appsignals-ocb-demo
318+
--platform-info ${{ env.CLUSTER_NAME }}
321319
--service-name sample-application-${{ env.TESTING_ID }}
322320
--remote-service-deployment-name ${{ env.REMOTE_SERVICE_DEPLOYMENT_NAME }}
321+
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
323322
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
324323
--rollup'
325-
# TODO: Undo hardcode
326-
# --platform-info ${{ env.CLUSTER_NAME }}
327-
# --region ${{ env.E2E_TEST_AWS_REGION }}
328324

329325
- name: Refresh AWS Credentials
330326
if: ${{ github.event.repository.name == 'aws-application-signals-test-framework' }}
@@ -353,14 +349,11 @@ jobs:
353349
run: |
354350
aws eks update-kubeconfig --name ${{ env.CLUSTER_NAME }} --region ${{ env.E2E_TEST_AWS_REGION }}
355351
eksctl delete iamserviceaccount --name cloudwatch-agent --namespace amazon-cloudwatch --cluster ${{ env.CLUSTER_NAME }} --region ${{ env.E2E_TEST_AWS_REGION }}
352+
kubectl delete -f ./appsignals-collector.yaml -n ${{ env.SAMPLE_APP_NAMESPACE }}
356353
kubectl delete ns opentelemetry-operator-system
357-
kubectl delete service appsignals-collector -n ${{ env.SAMPLE_APP_NAMESPACE }}
358-
kubectl delete service appsignals-collector-headless -n ${{ env.SAMPLE_APP_NAMESPACE }}
359-
kubectl delete service appsignals-collector-monitoring -n ${{ env.SAMPLE_APP_NAMESPACE }}
360354
kubectl delete deploy --all -n ${{ env.SAMPLE_APP_NAMESPACE }}
361355
kubectl delete service sample-remote-app-service -n ${{ env.SAMPLE_APP_NAMESPACE }}
362356
kubectl delete service sample-app-service -n ${{ env.SAMPLE_APP_NAMESPACE }}
363-
kubectl delete -f ./appsignals-collector.yaml -n ${{ env.SAMPLE_APP_NAMESPACE }}
364357
aws eks update-kubeconfig --name ${{ env.CLUSTER_NAME }} --region ${{ env.E2E_TEST_AWS_REGION }}
365358
366359
- name: Terraform destroy

0 commit comments

Comments
 (0)