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

Jump to bottom

mysql_user support REQUIRE NONE to remove a REQUIRE SSL "GRANT" #29811

Closed
ansibot opened this issue Sep 12, 2017 · 10 comments
Closed

mysql_user support REQUIRE NONE to remove a REQUIRE SSL "GRANT" #29811

ansibot opened this issue Sep 12, 2017 · 10 comments
Labels
affects_1.9 This issue/PR affects Ansible v1.9 bot_closed bug This issue/PR relates to a bug. collection:community.general collection Related to Ansible Collections work database Database category has_pr This issue has an associated PR. module This issue/PR relates to a module. mysql needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests.

Comments

@ansibot
Copy link
Contributor

ansibot commented Sep 12, 2017

From @drzraf on 2016-04-25T22:16:10Z

ISSUE TYPE
  • Bug Report
COMPONENT NAME

mysql_user

ANSIBLE VERSION
ansible 1.9.4
  configured module search path = /usr/share/ansible
CONFIGURATION
OS / ENVIRONMENT
SUMMARY

(originally posted as #15440)
There is no way to remove the "REQUIRE SSL" mysql privilege ... other than rewriting the whole set of permission which is not practicable (especially when depending upon append_privs=yes)

-  mysql_user: name=muser priv="*.*:REQUIRESSL" append_privs=yes

There is no way to revert such a directive. Normally this is done using REQUIRE NONE
But this is not implemented by the Ansible role.

STEPS TO REPRODUCE
-  mysql_user: name=muser priv="*.*:REQUIRENONE" append_privs=yes
EXPECTED RESULTS

The REQUIRE SSL is droped for "muser"

ACTUAL RESULTS

This is unsupported

Copied from original issue: ansible/ansible-modules-core#3542
@ansibot
Copy link
Contributor Author

ansibot commented Sep 12, 2017

From @ansibot on 2016-04-25T22:16:10Z

@Jmainguy ping, this issue is waiting for your response.
click here for bot help

@ansibot
Copy link
Contributor Author

ansibot commented Sep 12, 2017

From @Jmainguy on 2016-04-25T22:16:10Z

I would classify this as a feature to add and not a bug. That being said, if anybody wants to take a stab at working on this I would be glad to review the PR.

ansibot "needs_contributor"

@ansibot ansibot added the affects_1.9 This issue/PR affects Ansible v1.9 label Sep 12, 2017
@ansibot
Copy link
Contributor Author

ansibot commented Sep 12, 2017

From @drzraf on 2016-04-25T22:16:10Z

Are they many module provinding unrevertable operations (other than shell/command)?
It means state can't be changed afterwards.
Although I can't work on this myself, I'd say it's a bug anyway.

@ansibot ansibot mentioned this issue Sep 12, 2017
Closed
@ansibot ansibot added bug_report module This issue/PR relates to a module. support:community This issue/PR relates to code supported by the Ansible community. waiting_on_contributor This would be accepted but there are no plans to actively work on it. labels Sep 12, 2017
@ansibot ansibot added bug This issue/PR relates to a bug. and removed bug_report labels Mar 1, 2018
@Alexander198961 Alexander198961 mentioned this issue Jun 17, 2018
Closed
@ansibot ansibot added support:core This issue/PR relates to code supported by the Ansible Engineering Team. and removed support:community This issue/PR relates to code supported by the Ansible community. labels Sep 17, 2018
@ansibot ansibot added needs_maintainer Ansibot is unable to identify maintainers for this PR. (Check `author` in docs or BOTMETA.yml) support:community This issue/PR relates to code supported by the Ansible community. and removed support:core This issue/PR relates to code supported by the Ansible Engineering Team. labels Oct 3, 2018
@ansibot ansibot removed the needs_maintainer Ansibot is unable to identify maintainers for this PR. (Check `author` in docs or BOTMETA.yml) label Nov 10, 2018
@ansibot ansibot added the test This PR relates to tests. label Feb 3, 2019
@ansibot
Copy link
Contributor Author

ansibot commented Feb 19, 2019

cc @Xyon @bmalynovytch @michaelcoburn @Oneiroi @tolland
click here for bot help

@ansibot ansibot added the database Database category label Feb 19, 2019
@ansibot
Copy link
Contributor Author

ansibot commented Feb 27, 2019

cc @bmildren
click here for bot help

@Alexander198961 Alexander198961 mentioned this issue Mar 4, 2019
Closed
@ansibot
Copy link
Contributor Author

ansibot commented Jun 3, 2019

cc @Alexander198961
click here for bot help

@ansibot
Copy link
Contributor Author

ansibot commented Jun 19, 2019

cc @Andersson007
click here for bot help

@ansibot ansibot added the has_pr This issue has an associated PR. label Jul 30, 2019
@ansibot
Copy link
Contributor Author

ansibot commented Aug 8, 2019

cc @kurtdavis
click here for bot help

@ansibot
Copy link
Contributor Author

ansibot commented Dec 20, 2019

cc @tomaszkiewicz
click here for bot help

@ansibot ansibot added the collection Related to Ansible Collections work label Apr 29, 2020
@ansibot ansibot added collection:community.general needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md labels Apr 29, 2020
@ansibot ansibot added the needs_triage Needs a first human triage before being processed. label May 16, 2020
@ansibot
Copy link
Contributor Author

ansibot commented Aug 16, 2020

Thank you very much for your interest in Ansible. Ansible has migrated much of the content into separate repositories to allow for more rapid, independent development. We are closing this issue/PR because this content has been moved to one or more collection repositories.

For further information, please see:
https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md

@ansibot ansibot closed this as completed Aug 16, 2020
@sivel sivel removed the needs_triage Needs a first human triage before being processed. label Aug 17, 2020
@drzraf drzraf mentioned this issue Aug 23, 2020
Closed
@ansible ansible locked and limited conversation to collaborators Sep 13, 2020
@sivel sivel removed the waiting_on_contributor This would be accepted but there are no plans to actively work on it. label Dec 3, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
affects_1.9 This issue/PR affects Ansible v1.9 bot_closed bug This issue/PR relates to a bug. collection:community.general collection Related to Ansible Collections work database Database category has_pr This issue has an associated PR. module This issue/PR relates to a module. mysql needs_collection_redirect https://github.com/ansible/ansibullbot/blob/master/docs/collection_migration.md support:community This issue/PR relates to code supported by the Ansible community. test This PR relates to tests.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@sivel @dagwieers @ansibot