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

Add new parameters create_root_user and create_root_my_cnf. #578

Merged
merged 1 commit into from
Feb 9, 2015

Conversation

franzs
Copy link
Contributor

@franzs franzs commented Oct 6, 2014

This allows the galera module and others to write ${::root_home}/.my.cnf
independently from create the mysql user. This is useful for cluster
setups where you want to create ${::root_home}/.my.cnf on every node
but create the user only once.

This allows the galera module and others to write ${::root_home}/.my.cnf
independently from create the mysql user. This is useful for cluster
setups where you want to create ${::root_home}/.my.cnf on every node
but create the user only once.
@igalic
Copy link
Contributor

igalic commented Oct 7, 2014

@cmurphy looks good to me, what'd you think?

@vinzent
Copy link
Contributor

vinzent commented Nov 5, 2014

only applies to .my.cnf:

would't it make sense to put the puppet-mysql .my.cnf to for example $vardir/client_data/puppetlabs-mysql-my.cnf and use the --defaults-extra-file command line option?

so it would not interfere with others and IMHO reduce the risk of a human root user to make changes to mysql by accident.

@igalic
Copy link
Contributor

igalic commented Nov 7, 2014

heehehhe.. that's actually one of my favourite features ;)

I can just run: sudo -H mysql i'm logged in as mysql-root. :D


@vinzent but i see where you comin from. however, that would involve a deeper yak shave.

underscorgan pushed a commit that referenced this pull request Feb 9, 2015
Add new parameters create_root_user and create_root_my_cnf.
@underscorgan underscorgan merged commit b9fbba3 into puppetlabs:master Feb 9, 2015
@underscorgan
Copy link
Contributor

Thanks for the contribution @franzs !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants