-
Notifications
You must be signed in to change notification settings - Fork 225
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mssql tests are failing in CI because a recent mcr.microsoft.com/mssql/server
docker image release breaks our healthcheck
#4147
Comments
Yup... using CU13 (Cumulative Update) seems to get us working again. |
…kage 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
Having same issue here, thanks for reporting this |
+1, Having same issue as well |
…kage (#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
Have the same issue here! It is blocking us bigly (!), and we have an upcoming release. |
@MaxBeauchemin @alejandropadillav1 @Anonynym3845937 You say you are having the same issue. Do you mean you are having an issue with the I have worked around this issue for now by just not using the latest |
It looks like microsoft/mssql-docker#892 (comment) shows a working |
- move to the latest CU (cumulative update) of SQLServer 2022 (which involves updating the healthcheck for internal changes) - use MSSQL_SA_PASSWORD rather than the long since deprecated SA_PASSWORD Refs: #4147
Yes sorry, I am having an issue with the latest Docker image itself, but not in reference to this particular github repo using that image |
Hi, @trentm So, we had our pipeline configured with latest tag -> the error in Point 1 appeared. We patched it up and the error from Point 2 appeared. Since there was no answer at the time, we downgraded to the previous CU13 and all worked well. |
Is there a ticket on the sql's github open anywhere for this sql issue? |
@Anonynym3845937 See my comment above referring to microsoft/mssql-docker#892 (comment) |
- move to the latest CU (cumulative update) of SQLServer 2022 (which involves updating the healthcheck for internal changes) - use MSSQL_SA_PASSWORD rather than the long since deprecated SA_PASSWORD Refs: #4147
- move to the latest CU (cumulative update) of SQLServer 2022 (which involves updating the healthcheck for internal changes) - use MSSQL_SA_PASSWORD rather than the long since deprecated SA_PASSWORD Refs: #4147
…kage (elastic#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: elastic#4147
- move to the latest CU (cumulative update) of SQLServer 2022 (which involves updating the healthcheck for internal changes) - use MSSQL_SA_PASSWORD rather than the long since deprecated SA_PASSWORD Refs: elastic#4147
TAV=mssql tests are failing on
main
. E.g.:https://github.com/elastic/apm-agent-nodejs/actions/runs/10066834059/job/27830878023#step:3:167
This is because the healthcheck is failing:
apm-agent-nodejs/.ci/docker/docker-compose.yml
Line 45 in 921dbb9
That is because the command used for healthchecking,
/opt/mssql-tools/bin/sqlcmd
is now in a different dir:issue 2
However, if we update the path:
We get a failure (I'm guessing a different one now):
The go.microsoft.com link there is no use.
I stupidly don't know how to play with the older mssql/server iamge on my machine, even though I have its sha:
The docker hub page for this image (https://hub.docker.com/r/microsoft/mssql-server) confirms there was a recent release (earlier today):
Links in that Docker Hub page suggest the relevant repo for this image is https://github.com/microsoft/mssql-docker
However, there hasn't been a commit there in 4mo, no tags, no useful-looking branches. So either that repo is no longer used for releases or it hasn't been updated yet (i.e. is actually developed in private with periodic public syncs to this repo?).
There is no useful changelog that I can find.
There is this ancient issue asking for a working/workable Docker healthcheck for the image: microsoft/mssql-docker#133
I'm stuck here.
Perhaps there is an older tag we could pin to as a workaround for now.
The text was updated successfully, but these errors were encountered: