Skip to content

Commit c1aa525

Browse files
authored
ci: lock on older mssql/server Docker image to avoid healthcheck breakage (#4148)
The latest mcr.microsoft.com/mssql/server release (CU14) moves the sqlcmd install location, breaking our healthcheck. Just updating the path hits a different error. As a workaround, lets pin to the working mssql/server docker image for CI. Note that this docker compose file is used for TAV tests. I'm not sure if hte mssql/server image used by GH Actions' mssql 'service' will be affected at some point as well. Closes: #4147
1 parent 921dbb9 commit c1aa525

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

.ci/docker/docker-compose.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,9 @@ services:
3131
retries: 30
3232

3333
mssql:
34-
image: mcr.microsoft.com/mssql/server
34+
# Cumulative update 14 (CU14), released 2024-07-23, breaks the healthcheck.
35+
# See https://github.com/elastic/apm-agent-nodejs/issues/4147
36+
image: mcr.microsoft.com/mssql/server:2022-CU13-ubuntu-22.04
3537
platform: linux/amd64
3638
environment:
3739
- ACCEPT_EULA=Y

test/docker-compose.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,9 @@ services:
3131
retries: 30
3232

3333
mssql:
34-
image: mcr.microsoft.com/mssql/server
34+
# Cumulative update 14 (CU14), released 2024-07-23, breaks the healthcheck.
35+
# See https://github.com/elastic/apm-agent-nodejs/issues/4147
36+
image: mcr.microsoft.com/mssql/server:2022-CU13-ubuntu-22.04
3537
platform: linux/amd64
3638
environment:
3739
- ACCEPT_EULA=Y

0 commit comments

Comments
 (0)