Skip to content
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

MariaDB healthcheck changes in docker-compose sample #342

Open
simondarwent opened this issue Jun 24, 2023 · 3 comments · May be fixed by #426
Open

MariaDB healthcheck changes in docker-compose sample #342

simondarwent opened this issue Jun 24, 2023 · 3 comments · May be fixed by #426
Labels

Comments

@simondarwent
Copy link

With the recent update to MariaDB 11, the health check in the sample docker-compose leaves the container showing as unhealthy with an error output:

OCI runtime exec failed: exec failed: unable to start container process: exec: "mysqladmin": executable file not found in $PATH: unknown

Appears this is expected behaviour due to the removal of mysqladmin and suggests an alternative method to track container health using a healthcheck.sh script:

https://mariadb.org/mariadb-server-docker-official-images-healthcheck-without-mysqladmin/

@emijime
Copy link

emijime commented Jun 24, 2023

I have exactly the same issue from the previous update.

@couladin
Copy link

This is related to the tip given in #311, except the mysqladmin command no longer works. Suspect MariaDB removed that recently, but mariadb-admin does work properly.

In Docker, my Wallabag+MariaDB+Redis stack stopped working with a recent update, but altering the healthcheck line in my docker-compose file to the following makes it work perfectly:

test: ["CMD", "mariadb-admin" ,"ping", "-h", "localhost", "--password=<value_of_MYSQL_ROOT_PASSWORD_variable>"]

@j0k3r j0k3r added the bug label Jun 26, 2023
@GAS85
Copy link

GAS85 commented Apr 6, 2024

There is a solution in a #381

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants