Skip to content

Commit c4e83a6

Browse files
committed
undo hardcoded region
1 parent 77de498 commit c4e83a6

File tree

2 files changed

+8
-12
lines changed

2 files changed

+8
-12
lines changed

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

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -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,7 +273,7 @@ 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-east-1
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 }}
@@ -282,16 +284,14 @@ jobs:
282284
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
283285
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
284286
--rollup'
285-
# TODO: Undo hardcode
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-east-1
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 }}
@@ -303,16 +303,14 @@ jobs:
303303
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
304304
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
305305
--rollup'
306-
# TODO: Undo hardcodes:
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-east-1
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 }}
@@ -323,8 +321,6 @@ jobs:
323321
--remote-service-ip ${{ env.REMOTE_SERVICE_POD_IP }}
324322
--query-string ip=${{ env.REMOTE_SERVICE_POD_IP }}&testingId=${{ env.TESTING_ID }}
325323
--rollup'
326-
# TODO: Undo hardcode
327-
# --region ${{ env.E2E_TEST_AWS_REGION }}
328324

329325
- name: Refresh AWS Credentials
330326
if: ${{ github.event.repository.name == 'aws-application-signals-test-framework' }}

terraform/java/eks-otlp-ocb/util/appsignals-collector.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ spec:
5050
config: |
5151
extensions:
5252
sigv4auth:
53-
region: "us-east-1"
53+
region: "${E2E_TEST_AWS_REGION}"
5454
service: "xray"
5555
5656
receivers:
@@ -137,7 +137,7 @@ spec:
137137
138138
exporters:
139139
otlphttp:
140-
traces_endpoint: https://xray.us-east-1.amazonaws.com/v1/traces
140+
traces_endpoint: https://xray.${E2E_TEST_AWS_REGION}.amazonaws.com/v1/traces
141141
auth:
142142
authenticator: sigv4auth
143143

0 commit comments

Comments
 (0)