Skip to content
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

Backport of PRs to stable-2 #251

Merged

Conversation

Andersson007
Copy link
Collaborator

SUMMARY

Backport of #227, #239, #240, #248

koleo and others added 4 commits November 29, 2021 10:40
The naming rules for MySQL/MariaDB identifiers, when quoted, allow the
`%` character.

However, currently, the use of the `%` character in database names
results in mismatch or missing databases.

- Rewrite query to identify the databases in the catalog using
  `information_schema` instead of `SHOW DATABASES LIKE`
- Escape the `%` character in `CREATE DATABASE` query.

Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit 6b12435)
Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit 5522e45)
- Define variables "db_names" and "db_formats" in defaults
- Use of the "vars" option in includes instead of default parameters
  that might be overridden by a previous task
- Use of the "loop" option in includes instead of duplicating include
  tasks
- Use a nested loop on db_names and db_formats in state_dump_import test

Signed-off-by: Nicolas Payart <[email protected]>
(cherry picked from commit e4de13a)
@codecov
Copy link

codecov bot commented Nov 29, 2021

Codecov Report

Merging #251 (8409c51) into stable-2 (1cb39cc) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##           stable-2     #251   +/-   ##
=========================================
  Coverage     77.61%   77.61%           
=========================================
  Files            24       24           
  Lines          2171     2171           
  Branches        510      510           
=========================================
  Hits           1685     1685           
  Misses          319      319           
  Partials        167      167           
Impacted Files Coverage Δ
plugins/modules/mysql_db.py 74.56% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1cb39cc...8409c51. Read the comment docs.

@Andersson007 Andersson007 merged commit 7aab5cc into ansible-collections:stable-2 Nov 29, 2021
@Andersson007
Copy link
Collaborator Author

Backport, no review is needed

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.

2 participants