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

fix mysql config read host (.my.cnf) #6761 #37400

Open
wants to merge 1 commit into
base: devel
from

Conversation

Projects
None yet
5 participants
@genofire

genofire commented Mar 14, 2018

let mysqllib choose default values)

SUMMARY
ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

every mysql_* module

ANSIBLE VERSION
ansible 2.4.3.0
  config file = /etc/ansible/ansible.cfg
  configured module search path = [u'/home/genofire/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/lib/python2.7/site-packages/ansible
  executable location = /usr/sbin/ansible
  python version = 2.7.14 (default, Jan  5 2018, 10:41:29) [GCC 7.2.1 20171224]
ADDITIONAL INFORMATION

see issue


@ansibot

This comment has been minimized.

Contributor

ansibot commented Mar 14, 2018

@ansibot

This comment has been minimized.

Contributor

ansibot commented Mar 14, 2018

The test ansible-test sanity --test validate-modules [explain] failed with 2 errors:

lib/ansible/modules/database/mysql/mysql_user.py:0:0: E324 Value for "default" from the argument_spec (None) for "login_host" does not match the documentation ('localhost')
lib/ansible/modules/database/mysql/mysql_variables.py:0:0: E324 Value for "default" from the argument_spec (None) for "login_host" does not match the documentation ('localhost')

click here for bot help

@genofire

This comment has been minimized.

genofire commented May 30, 2018

@mpdehaan ??

@genofire

This comment has been minimized.

genofire commented Jul 17, 2018

Hey whats up @Jmainguy and @banyek - a new ansible version, where i have to fix this issue again (on my daily ansible)

@seanorama

This comment has been minimized.

seanorama commented Jul 30, 2018

Any plans to accept this in? Seems odd that ~/.my.cnf is not being respected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment