Deprecated mysql name when using mariadb #727

Closed
opened 2023-07-22 08:24:15 +00:00 by nsurbay · 3 comments

What I'm trying to do and why

The commit b30b040b73 deprecates the usage of mysql and mysql-* when using mariadb database.

When using a borgmatic configuration with mysql_databases, the followed message is displayed:

mysql: Deprecated program name. It will be removed in a future release, use '/usr/bin/mariadb' instead

Steps to reproduce

Run borgbackup to backup an mariadb database using mysql_databases options. The mariadb server and CLI should be at least at version 11.0.2.

Actual behavior

mysql: Deprecated program name. It will be removed in a future release, use '/usr/bin/mariadb' instead

Expected behavior

No warning is displayed and borgbackup use mariadb instead of mysql.

Other notes / implementation ideas

As you may want to support both backend, maybe a new option in mysql_databases can be use to select either mysql or mariadb. Another implementation may be copy mysql_databases to a mariadb_databases.

borgmatic version

1.8.0

borgmatic installation method

archlinux package

Borg version

borg 1.2.4

Python version

Python 3.11.3

Database version (if applicable)

mariadb from 11.0.2-MariaDB, client 15.2 for Linux (x86_64) using readline 5.1

Operating system and version

archlinux

### What I'm trying to do and why The commit https://github.com/MariaDB/server/commit/b30b040b733ff2045ffcd7bdd44f608c7f4912b5 deprecates the usage of ``mysql`` and ``mysql-*`` when using mariadb database. When using a borgmatic configuration with ``mysql_databases``, the followed message is displayed: ``` mysql: Deprecated program name. It will be removed in a future release, use '/usr/bin/mariadb' instead ``` ### Steps to reproduce Run borgbackup to backup an mariadb database using ``mysql_databases`` options. The mariadb server and CLI should be at least at version 11.0.2. ### Actual behavior ``` mysql: Deprecated program name. It will be removed in a future release, use '/usr/bin/mariadb' instead ``` ### Expected behavior No warning is displayed and borgbackup use mariadb instead of mysql. ### Other notes / implementation ideas As you may want to support both backend, maybe a new option in ``mysql_databases`` can be use to select either mysql or mariadb. Another implementation may be copy ``mysql_databases`` to a ``mariadb_databases``. ### borgmatic version 1.8.0 ### borgmatic installation method archlinux package ### Borg version borg 1.2.4 ### Python version Python 3.11.3 ### Database version (if applicable) mariadb from 11.0.2-MariaDB, client 15.2 for Linux (x86_64) using readline 5.1 ### Operating system and version archlinux
Owner

Thank you for bringing this to my attention! My inclination here is to do your second suggestion (a separate mariadb_databases:) for a couple reasons:

  • mysql_databases: is a list! So it would require a more involved schema change to be able to stuff an option in there to select MySQL/MariaDB.
  • Based on the MariaDB changeset you linked, it's sounding like MariaDB commands may diverge further from MySQL's in the future. In which case it would make sense for borgmatic to have its own dedicated MariaDB hook in order to support those differences.
Thank you for bringing this to my attention! My inclination here is to do your second suggestion (a separate `mariadb_databases:`) for a couple reasons: * `mysql_databases:` is a list! So it would require a more involved schema change to be able to stuff an option in there to select MySQL/MariaDB. * Based on the MariaDB changeset you linked, it's sounding like MariaDB commands may diverge further from MySQL's in the future. In which case it would make sense for borgmatic to have its own dedicated MariaDB hook in order to support those differences.
Owner

I just implemented a new mariadb_databases: hook in main and it'll be part of the next release! Be aware though that any existing backups made with the mysql_databases: hook are only restorable with a mysql_databases: configuration. Thanks for the suggestion!

I just implemented a new `mariadb_databases:` hook in main and it'll be part of the next release! Be aware though that any existing backups made with the `mysql_databases:` hook are only restorable with a `mysql_databases:` configuration. Thanks for the suggestion!
Owner

This has been released in borgmatic 1.8.2!

This has been released in borgmatic 1.8.2!
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: borgmatic-collective/borgmatic#727
No description provided.