Skip to content

Fix CentOS #788

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

Closed
wants to merge 3 commits into from
Closed

Conversation

elconas
Copy link

@elconas elconas commented Dec 17, 2015

Fixed CentOS (not nice, but using confines and changing the whole class hierarchy is too complex for me). Using has_command { environment :PATH => .... } does not work too ... so this hack "is it" :)

Robert Heinzmann added 3 commits October 13, 2015 08:52
  - Added MySQL version and flavour detection support
  - Added mysql_datadir provider/type (replaces Exec[mysql_install_db])
  - Added version specific parameters my.cnf ([mysqld-5.X] sections)
  - Version specific user mangement SQL (ALTER USER for 5.7.6++ ...)
@elconas
Copy link
Author

elconas commented Dec 17, 2015

Hi, this failed on CentOS with default MySQL (non extra repos) as /usr/libexec was not part of the path for mysqld.
Fixed in #788 and tested on CentOS with default package and upstream mysql-community packages.
Tested on:

CentOS6:

  • MariaDB 10
  • Oracle MySQL Community Edition 5.5 (mysql-community-server)
  • Oracle MySQL Community Edition 5.6 (mysql-community-server)
  • Oracle MySQL Community Edition 5.7 (mysql-community-server)
  • CentOS default mysql 5.1 (mysql-server)

Also Updated https://tickets.puppetlabs.com/browse/MODULES-2676

@igalic
Copy link
Contributor

igalic commented Dec 17, 2015

@elconas could you do this as a proper rebase here?
see also voxpupuli/puppet-jira#99 (comment) for how to do that

@elconas
Copy link
Author

elconas commented Dec 17, 2015

Created new Pull Request #789

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

Successfully merging this pull request may close these issues.

2 participants