Bug #11375: puppetlabs-mysql fails on CentOS/RHEL #27
Conversation
| @@ -44,6 +44,11 @@ | |||
| ensure => directory, | |||
| mode => '755', | |||
| } | |||
| file { '/etc/mysql/conf.d': | |||
| ensure => directory, | |||
| require => File['/etc/mysql'], | |||
There was a problem hiding this comment.
This require isn't necessary. Implicit dependencies will take care of this.
|
@ccaum - good catch, require removed. @ghoneycutt - I'm not seeing conf.d on any RH systems, but this module tries to include that dir via my.cnf.erb. My commit was an attempt to fix that. However, that got me digging, and I think I've found another problem. At least on my CentOS 6 systems, the mysql-server rpm doesn't setup /etc/mysql either, the binary is looking for /etc/my.cnf. So, I think we should maybe add a $config_file var to params.pp to do this all a bit more intelligently perhaps? Is this a separate issue altogether? |
|
I'm in #puppet as justintime for the next hour if you need me real-time. |
|
Would it make sense to symlink /etc/my.cnf to /etc/mysql/my.cnf on CentOS 6 systems? |
|
The only problem is that I'm pretty sure /etc/my.cnf and /etc/mysql/my.cnf differ on CentOS and Ubuntu. What about using a variable in params.pp and using file_line() from stdlib to just make sure that our /etc/mysql/conf.d/ directory is included? |
|
Umm, yuck: http://pastebin.com/mcK3U1XH That's a diff of CentOS6 vs Debian 6.0.2 |
Bug #11375: puppetlabs-mysql fails on CentOS/RHEL Reviewed by Carl Caum <carl@puppetlabs.com>
…l-task-management (MODULES-6800) Add Windows service management
https://projects.puppetlabs.com/issues/11375