Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

Allow configurable path to service #13

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tdooner
Copy link

@tdooner tdooner commented Jul 26, 2014

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!

When trying to use this cookbook on a CentOS host, the path to the
service executable is different so SmartStack cannot restart haproxy.
@spuder
Copy link

spuder commented May 1, 2015

Looks like this has been open for a while. Anything stopping this from being merged?

@tdooner
Copy link
Author

tdooner commented May 1, 2015

@spuder It seems to me that this repo is basically dead. My company, Brigade, maintains a fork at https://github.com/brigade/smartstack-cookbook which works for us in our rbenv / CentOS 7 environment. You're welcome to give it a try if you want, but YMMV.

@spuder
Copy link

spuder commented May 1, 2015

Thanks for letting me know. I'm also evaluating consul. It appears to do the same thing.

@tdooner
Copy link
Author

tdooner commented May 1, 2015

We're also looking into that. FWIW, even if we end up using Consul, I think it's likely we'll continue using synapse. It's really good at interacting with haproxy.

Fortunately, synapse has an abstraction around the backends it supports, so it should be possible to write a consul watcher as others have done for Etcd and Marathon (we're running that marathon PR in production)

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

Successfully merging this pull request may close these issues.

2 participants