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

Allow multiple MySQL instances #267

Merged
merged 4 commits into from
Dec 19, 2016
Merged

Allow multiple MySQL instances #267

merged 4 commits into from
Dec 19, 2016

Conversation

truthbk
Copy link
Member

@truthbk truthbk commented Dec 2, 2016

Rebases #262

Thanks @IanCrouch!

@italicninja
Copy link
Contributor

@truthbk I can't figure out the Travis errors it's throwing for the life of me. In my test environment it does what's expected. Is this the case for you?

@truthbk
Copy link
Member Author

truthbk commented Dec 16, 2016

@IanCrouch I'm hitting issues with puppet versions <4.x as well. Haven't had a whole lot of time to check it out. We'll get there soon enough :)

@truthbk
Copy link
Member Author

truthbk commented Dec 19, 2016

Finally got these tests fixed. Merging! Thanks @IanCrouch, that undef was the problem with the 3.x puppets.

@truthbk truthbk merged commit 060f111 into master Dec 19, 2016
@truthbk truthbk deleted the jaime/mysql_instance branch December 19, 2016 18:03
cegeka-jenkins pushed a commit to cegeka/puppet-datadog_agent that referenced this pull request Jan 31, 2018
* Allowing Multiple MySQL instances

* Tweaking tags and Dealing with undef sock declarations

* Trying to fix Travis Failures involving the sock declaration

* [puppet] address issue with undefined sock parameter.
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.

None yet

3 participants