Conversation
|
@mrsdizzie @schlessera curious to hear your thoughts |
Codecov ReportAttention: Patch coverage is
📢 Thoughts on this report? Let us know! |
|
Maybe it could be good to have one of the test runs use Mariadb instead of MySQL as well? |
|
Yes, see wp-cli/.github#123 Feel free to submit a PR :) |
Because we need these both in this project and in wp-cli-tests
|
It seems like this change may have introduced a hiccup in our workflow with wp-cli and mariadb. in my wordpress docker container i'm running Debug (db): Final MySQL command: /usr/bin/env mariadb --no-defaults --no-auto-rehash --batch --skip-column-names --host='db' --user='wordpress_user' --default-character-set='utf8' --execute='SELECT @@SESSION.sql_mode' (0.061s) I dug a little deeper to find a little more debugging info. running the command diretly shows the same error i read adding the arg from there, it does work if i also add the "--password=blah" arg. Has anyone else overcome this in their workflow with docker+mariadb+wp-cli and can offer help? |
Symptom: Database transfers started failing all of the sudden, claiming `mariadb-dump` could not be found. Reason: wp-cli/wp-cli#6072
See:
MariaDB currently adds symlinks for
mysqlto usemariadbunder the hood, but that is being deprecated and people get warnings likeDeprecated program name. It will be removed in a future release, use 'mariadb' insteadWith this change, the idea is to offer better utils for commands like db-command to pick the right binary.
I suppose that could imply that we officially support MariaDB, but I guess since WordPress core itself recommends it, we should at least test it.