Skip to content

Mysqlpustream #1

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

Draft
wants to merge 22 commits into
base: main
Choose a base branch
from
Draft

Mysqlpustream #1

wants to merge 22 commits into from

Conversation

minorOffense
Copy link
Member

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.

Checklist

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

rajat-puppet and others added 22 commits July 22, 2024 17:31
sha_of_the_head_commit
Value of the facter :macaddress was nil, causing tests to have two
failures (contexts "igalic's laptop" and "node with lo only").

Update the facter initialization allows to pass the tests
successfully.
fix mistakes

fix spec for the changed query
Fix spec where mysql_users was failing for SLES
* When using multiple excluded databases, the list of databases
  is filtered using `grep -v`.
  i.e. `grep -v '^\(information_schema|performance_schema\)$`
* When using Basic vs Extended Regular Expressions, the characters
  `(` and `|` lose their special meaning, the backslashed versions
  have to be used.
  For the group (`()`) the escaping has been done, however the
  alternation is unescaped.

Leading to:

* All the excluded databases will be backed up.
* In case a database is not backuppable (which is why it had been
  excluded), this leads to the cleanup not being run at all, as it
  depends on the backup having been successful.

This MR aims to fix this issue, by revising the regular expression
and specifying that behaviour in the respective class spec.
Fix backup/rotation with multiple excluded databases
pdksync - (PF-3525) - pdk update for module
 - Add all Debian 11 and above platforms to have expected privileges.
 - Add ruby package for Debian 12
 - When downloading the mariadb-server package '/var/log/mysql' is automatically created for other platforms, but that is not the case with Debian 12. So, make sure the directory is present.
 - Debian 12 uses utf8mb3 encoding since utf8 is not supported on it.
 - Skip Debian-12-arm platform from tests since the mysql binaries, 'my_print_defaults' and 'mysql_config_editor', are not supported for ARM architecture.
…n-12-support

(CAT-2100) Add Debian 12 support
sha_of_the_head_commit
Force mariadb on everything
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.