From 1a326b98ce55576cc1d8759aea4742aaa6061f9c Mon Sep 17 00:00:00 2001 From: Trent Mick Date: Wed, 24 Jul 2024 08:17:14 -0700 Subject: [PATCH] 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: https://github.com/elastic/apm-agent-nodejs/issues/4147 --- .ci/docker/docker-compose.yml | 4 +++- test/docker-compose.yml | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/.ci/docker/docker-compose.yml b/.ci/docker/docker-compose.yml index cf1e6555b0..4ecdf630d4 100644 --- a/.ci/docker/docker-compose.yml +++ b/.ci/docker/docker-compose.yml @@ -31,7 +31,9 @@ services: retries: 30 mssql: - image: mcr.microsoft.com/mssql/server + # Cumulative update 14 (CU14), released 2024-07-23, breaks the healthcheck. + # See https://github.com/elastic/apm-agent-nodejs/issues/4147 + image: mcr.microsoft.com/mssql/server:2022-CU13-ubuntu-22.04 platform: linux/amd64 environment: - ACCEPT_EULA=Y diff --git a/test/docker-compose.yml b/test/docker-compose.yml index a87e4435c3..5ee22bc6f7 100644 --- a/test/docker-compose.yml +++ b/test/docker-compose.yml @@ -31,7 +31,9 @@ services: retries: 30 mssql: - image: mcr.microsoft.com/mssql/server + # Cumulative update 14 (CU14), released 2024-07-23, breaks the healthcheck. + # See https://github.com/elastic/apm-agent-nodejs/issues/4147 + image: mcr.microsoft.com/mssql/server:2022-CU13-ubuntu-22.04 platform: linux/amd64 environment: - ACCEPT_EULA=Y