added helper method in mysql provider to pick mysql or mariadb binary…

[rtuk]

added helper method in mysql provider to pick mysql or mariadb binary variant based on version

changed commands to use the new helper method

Summary

Provide a detailed description of all the changes present in this pull request.

Additional Context

Add any additional context about the problem here.

• Root cause and the steps to reproduce. (If applicable)
• Thought process behind the implementation.

Related Issues (if any)

Mention any related issues or pull requests.
#1685
#1676
#1672

Checklist

• 🟢 Spec tests.
• 🟢 Acceptance tests.
• Manually verified. (For example puppet apply)

[CLAassistant]

All committers have signed the CLA.

[j-koehler]

Wow, looks like Perforce is not interested in any community work anymore and putting the whole Puppet eco system behind closed doors. Their "official" puppetlabs-mysql is broken for any supported MariaDB release, this PR (presumably) fixes this and is still "open" for months. Wow.

[SimonHoenscheid]

@jst-cyr can perforce please fix the errors with MariaDB 11.x? That is laying around for quite some time and needs a release afterwards. See also the linked issues.

[jst-cyr]

Thank you for flagging this to me @SimonHoenscheid . I'm going to add this particular PR into my list to review with engineering, but it looks like there is a general need for some mysql updates here which I'll also need to review.

[SimonHoenscheid]

@jst-cyr I am not exactly sure this PR will solve the problem. But the issue linked needs to be solved.

Thanks for having a look on this.

[cruelsmith]

JFYI: A simple remove of the hard coded commands inside the child provider files did also fixed the issue, since #1645 already added the handling in the parent provider file.

Requried cleanup bc7fc80