fix: Replace mysql with mariadb binary in MariaDB module

[FlorianHockmann]

The command has been renamed from mysql to mariadb and it looks like the symlink that has been in place for many versions has also been removed in newer MariaDB versions.

What does this PR do?

mysql command changed to mariadb for the MariaDB module.

Why is it important?

Wait policy fails otherwise for newer MariaDB images.

Related issues

• Closes [Bug]: MariaDB WaitUntil expects mysql command, replaced by mariadb in newer version #941

How to test this PR

Try to use the MariaDB image with a new tag, like latest.

[netlify]

✅ Deploy Preview for testcontainers-dotnet ready!

Name	Link
🔨 Latest commit	7e6a7e9
🔍 Latest deploy log	https://app.netlify.com/sites/testcontainers-dotnet/deploys/649e71b0c452c80008b1a1cf
😎 Deploy Preview	https://deploy-preview-942--testcontainers-dotnet.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[HofmeisterAn]

Thank you for raising the issue and submitting the PR. I think we need to update the builder too:

testcontainers-dotnet/src/Testcontainers.MariaDb/MariaDbBuilder.cs

Line 138 in c8a44a7

_command = new List<string> { "mysql", "--protocol=TCP", $"--port={MariaDbPort}", $"--user={configuration.Username}", $"--password={configuration.Password}", configuration.Database, "--wait", "--silent", "--execute=SELECT 1;" };

Do you know if removing the symlink is intended (are there any release notes)? Maybe it is an issue in the latest version of the image. I assume this will break many configurations, see: MariaDB/mariadb-docker#519.

[FlorianHockmann]

I think we need to update the builder too:

Oh right, I just updated the commit to also change it there.

Do you know if removing the symlink is intended (are there any release notes)?

I only found this part in the release notes:

MDEV-29582 deprecate mysql* names

but looks like they didn't only deprecate them...

It however sounds me like they don't want to revert this change:

I haven't see a major case for re-adding mysql commands, we where going to do it eventually, maybe a depreciation would have been nicer, but it seems to be done and the major use case of healthchecks has been improved.

(Source)

But at least for Testcontainers it makes sense in my opinion to switch to the mariadb command even if they would be adding the symlinks back as mariadb is the correct name and the symlink was only for backwards compatibility.