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

#11963 In the mysql module the Exec[mysqld-restart] should have more in path #42

Merged
merged 2 commits into from
Mar 16, 2012

Conversation

fcharlier
Copy link
Contributor

As seen in the issue (https://projects.puppetlabs.com/issues/11963), there's already a fix waiting in another stalled pull-request containing other fixes.

Maybe this issue can be resolved separately ?

@bodepd
Copy link
Contributor

bodepd commented Mar 15, 2012

can you rebase this off the latest master? I just merged some changes that have merge conflicts with this pull request

And log output on failure.

Signed-off-by: François Charlier <fcharlier@ploup.net>
@fcharlier
Copy link
Contributor Author

Code rebased !

And fix a coma error in mysql::config class.

Signed-off-by: François Charlier <fcharlier@ploup.net>
@fcharlier
Copy link
Contributor Author

Updated (about note at fcharlier@51a5f61#commitcomment-1094965)

@bodepd
Copy link
Contributor

bodepd commented Mar 16, 2012

@fcharlier looks great!

bodepd added a commit that referenced this pull request Mar 16, 2012
#11963 In the mysql module the Exec[mysqld-restart] should have more in path
@bodepd bodepd merged commit a3ce533 into puppetlabs:master Mar 16, 2012
pmcmaw pushed a commit to pmcmaw/puppetlabs-mysql that referenced this pull request Mar 3, 2021
…-0-g34e3266

pdksync - Update using 1.5.0
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.

3 participants