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

Support for exposing Service data in KV store? - (consul/confd integration) #280

Closed
johnrengelman opened this issue Aug 11, 2014 · 5 comments

Comments

@johnrengelman
Copy link

So a little background. We were looking to use Consul to do service discovery in our architecture and we wanted to use it to configure the backend servers for a number of HAProxy instances. For this, we were looking to use confd. However confd only integrates with the KV store in consul.

We tried to add a feature to support exposing the /v1/health endpoint data to confd but after long discussion it was ultimately rejected as it strayed too far from the base concepts of confd (kelseyhightower/confd#102).

So my question is...is there any interest or possibility of enabling a mode where the service health information is exposed automatically in the KV store in consul? Is it a feature that if provided via PR, would be accepted? Or is it something that is just too different from the core concepts of consul.

@mohitarora
Copy link

+1 for this feature. We are also using confd. It will be great if services are available via K/V store.

@salehe
Copy link

salehe commented Aug 11, 2014

+1. It's been a long time that I'm thinking of requesting this. Thanks @johnrengelman

@armon
Copy link
Member

armon commented Aug 11, 2014

We just blogged about our tool consul-haproxy https://github.com/hashicorp/consul-haproxy here: http://www.hashicorp.com/blog/haproxy-with-consul.html.

While I wrote this specifically thinking about HAProxy, I think there is room for us to fork this and make it more generic.

@johnrengelman
Copy link
Author

Perfect...That actually should cover our use case ATM, but some general abstraction would be great!

@armon
Copy link
Member

armon commented Aug 22, 2014

Closing since consul-haproxy handles this

@armon armon closed this as completed Aug 22, 2014
duckhan pushed a commit to duckhan/consul that referenced this issue Oct 24, 2021
* syncCatalog: add `addK8SNamespaceSuffix` property and set it to true by default.
   This is a breaking change because an upgrade to this version will unregister all services 
   in Consul and register them with a new name.
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

No branches or pull requests

4 participants