This repository has been archived by the owner on Jul 29, 2020. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello! First, thanks for this awesomely configurable cookbook. It has made it possible for me to find the joy of smartstack without needing to configure barely anything to begin with.
I know this cookbook is intended for Ubuntu audiences only, but we're looking at using it on CentOS. Only one snag to be found (so far?): the path to the service executable is different so SmartStack cannot restart haproxy.
This PR is a quick hack allowing a different path based on node['platform'], and for other platforms to be able to configure it via an attribute. I'd be happy to rewrite a different way if you think of a better way.
Also, is there a convenient way to run the test suite against this? I did some minimal verification locally but some continuous integration here could go a long way.
Thanks!