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

Database healthcheck throws warning #51

Open
yorkshire-pudding opened this issue May 2, 2024 · 1 comment
Open

Database healthcheck throws warning #51

yorkshire-pudding opened this issue May 2, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@yorkshire-pudding
Copy link

yorkshire-pudding commented May 2, 2024

I have a Lando recipe and GitHub action that runs tests. I've just updated it to use the lando/setup-lando@v3 gha (which works a treat btw)

Recipe: Backdrop
Lando version: "latest"
Database: mysql

When Lando is started, it mostly runs fine, but when it does the database health check this warning appears:
Warning: LED] mysql: [Warning] Using a password on the command line interface can be insecure.

I wonder if the healthcheck could switch to using a .cnf file for the creds to avoid this warning?

I'm also getting a failed healthcheck locally for the same reason even though the database works otherwise.

@yorkshire-pudding yorkshire-pudding added the bug Something isn't working label May 2, 2024
@yorkshire-pudding yorkshire-pudding changed the title Database healthcheck on GitHub Action throws warning Database healthcheck throws warning May 3, 2024
@yorkshire-pudding
Copy link
Author

What I've done to avoid this: in my .lando.yml file, override the health check, thus:

services:  
  database:
    healthcheck: mysql --defaults-extra-file=/app/.lando/mysql.cnf --silent --execute "SHOW DATABASES;"

and create in /app/.lando/mysql.cnf

[client]
  user=backdrop
  password=backdrop
  host=database

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant