Skip to content

Commit 129d5c6

Browse files
authored
Do not use a connection string for SQS container test (#4566)
* Do not use a connection string for SQS container test * Add the env vars
1 parent b18fc79 commit 129d5c6

File tree

2 files changed

+6
-3
lines changed

2 files changed

+6
-3
lines changed

.github/workflows/container-integration-test.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ jobs:
3737
expected-healthy-containers: 4
3838
- name: sqs
3939
transport: AmazonSQS
40-
connection-string: 'AccessKeyId=demo;SecretAccessKey=demo;Region=us-east-1'
40+
# No connection-string - should be able to load AWS info only from environment variables
4141
compose-cmd: docker compose -f servicecontrol.yml -f sqs.yml up -d
4242
expected-healthy-containers: 5
4343
env:
@@ -127,4 +127,4 @@ jobs:
127127
echo "Found $numHealthy healthy containers out of ${{matrix.expected-healthy-containers}} expected"
128128
if ($numHealthy -ne ${{matrix.expected-healthy-containers}}) {
129129
exit 1
130-
}
130+
}
Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,7 @@
11
SERVICECONTROL_TAG=$SERVICECONTROL_TAG
22
TRANSPORTTYPE=$TRANSPORTTYPE
33
CONNECTIONSTRING="$CONNECTIONSTRING"
4-
AWS_ENDPOINT_URL=http://localstack:4566
4+
AWS_ENDPOINT_URL=http://localstack:4566
5+
AWS_ACCESS_KEY_ID=demo
6+
AWS_SECRET_ACCESS_KEY=demo
7+
AWS_REGION=us-east-1

0 commit comments

Comments
 (0)