Skip to content

Commit e75c380

Browse files
author
Janne Rönkkö
committed
Do not require sqlcmd to be installed on the GHA runner when setting up e2e environment
1 parent 09c558a commit e75c380

1 file changed

Lines changed: 14 additions & 2 deletions

File tree

github-actions/setup-e2e-environment/action.yml

Lines changed: 14 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -172,9 +172,21 @@ runs:
172172
env:
173173
SA_PASSWORD: "P@ssw0rd"
174174
with:
175+
# The IP address of MS SQL Docker container is retrieved using docker inspect because the container
176+
# does not have a predefined or predictable hostname. A predefined hostname could be added in docker
177+
# compose bundle.
175178
command:
176-
'sqlcmd -S 127.0.0.1 -U sa -P "$SA_PASSWORD" -d master -Q "SELECT
177-
''OK'';"'
179+
docker run
180+
--rm
181+
--network docker_jore4 \
182+
$(docker inspect mssqltestdb | jq -r '.[0].Config.Image')
183+
/opt/mssql-tools18/bin/sqlcmd
184+
-C
185+
-S $(docker inspect mssqltestdb | jq -r '.[0].NetworkSettings'.Networks.docker_jore4.IPAddress)
186+
-U sa
187+
-P "$SA_PASSWORD"
188+
-d master
189+
-Q "SELECT ''OK'';"
178190
# it takes a while for the database to start
179191
retries: 50
180192

0 commit comments

Comments
 (0)