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

mysql_replication: Add aliases to "master" options, add alternatives to "master" state options, add announcement #150

Merged

Conversation

Andersson007
Copy link
Collaborator

SUMMARY

Relates to #145

mysql_replication:

  1. add aliases to "master_" options
  2. add alternatives to "master" choices of state options
  3. add deprecation warnings and announcement

Important: this does NOT contain any breaking changes. I ran the not touched tests with the changes - they passed. Then changed, the changed ones also passed (but you can recheck if any doubts, just checkout to the first commit of the PR).

@codecov
Copy link

codecov bot commented Apr 16, 2021

Codecov Report

Merging #150 (14b19d4) into main (853db5a) will decrease coverage by 2.19%.
The diff coverage is 61.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   79.11%   76.92%   -2.20%     
==========================================
  Files          20       20              
  Lines        3194     1794    -1400     
  Branches      815      439     -376     
==========================================
- Hits         2527     1380    -1147     
+ Misses        412      268     -144     
+ Partials      255      146     -109     
Impacted Files Coverage Δ
plugins/modules/mysql_replication.py 65.82% <61.42%> (-3.17%) ⬇️
...ugins/module_utils/implementations/mariadb/user.py 77.77% <0.00%> (-22.23%) ⬇️
plugins/modules/mysql_info.py 79.69% <0.00%> (-2.60%) ⬇️
plugins/modules/mysql_variables.py 87.28% <0.00%> (-1.77%) ⬇️
plugins/modules/mysql_query.py 89.02% <0.00%> (-0.98%) ⬇️
plugins/modules/mysql_db.py 74.56% <0.00%> (-0.69%) ⬇️
tests/unit/plugins/modules/test_mysql_info.py 100.00% <0.00%> (ø)
tests/unit/plugins/modules/test_mysql_user.py 100.00% <0.00%> (ø)
...ins/module_utils/test_mysql_user_implementation.py 100.00% <0.00%> (ø)
... and 8 more

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 853db5a...14b19d4. Read the comment docs.

@Andersson007 Andersson007 changed the title [WIP] mysql_replication: Add aliases to "master" options, add alternatives to "master" state options, add announcement mysql_replication: Add aliases to "master" options, add alternatives to "master" state options, add announcement Apr 16, 2021
@Andersson007
Copy link
Collaborator Author

CC @Jorge-Rodriguez @bmalynovytch

plugins/modules/mysql_replication.py Outdated Show resolved Hide resolved
plugins/modules/mysql_replication.py Outdated Show resolved Hide resolved
Andersson007 and others added 2 commits April 19, 2021 09:18
Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>
Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>
@Andersson007
Copy link
Collaborator Author

@Jorge-Rodriguez thanks, committed, ready for review

@Andersson007 Andersson007 merged commit c8f9b1c into ansible-collections:main Apr 20, 2021
@Andersson007
Copy link
Collaborator Author

@Jorge-Rodriguez @bmalynovytch thanks for reviewing!

Andersson007 added a commit to Andersson007/community.mysql that referenced this pull request Apr 23, 2021
…to "master" state options, add announcement (ansible-collections#150)

* mysql_replication: Add aliases, add alternatives for the state option, announce major changes

* Change tests

* Add changelog fragment

* Fix changelog

* Update plugins/modules/mysql_replication.py

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>

* Update plugins/modules/mysql_replication.py

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>
(cherry picked from commit c8f9b1c)
Andersson007 added a commit that referenced this pull request Apr 23, 2021
…to "master" state options, add announcement (#150) (#152)

* mysql_replication: Add aliases, add alternatives for the state option, announce major changes

* Change tests

* Add changelog fragment

* Fix changelog

* Update plugins/modules/mysql_replication.py

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>

* Update plugins/modules/mysql_replication.py

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>

Co-authored-by: Jorge Rodriguez (A.K.A. Tiriel) <[email protected]>
(cherry picked from commit c8f9b1c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants