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

Add an agent API to manage watches #1918

Open
slackpad opened this issue Apr 5, 2016 · 6 comments
Open

Add an agent API to manage watches #1918

slackpad opened this issue Apr 5, 2016 · 6 comments
Labels
theme/api Relating to the HTTP API interface type/enhancement Proposed improvement or new feature
Milestone

Comments

@slackpad
Copy link
Contributor

slackpad commented Apr 5, 2016

Capturing this after the discussion here - https://groups.google.com/d/msgid/consul-tool/01b167e3-be26-4bcb-ba61-0836bcd1e3c6%40googlegroups.com?utm_medium=email&utm_source=footer.

@slackpad slackpad added the type/enhancement Proposed improvement or new feature label Apr 5, 2016
@fdhex
Copy link

fdhex commented Apr 20, 2016

I would actually very much support this.

@slackpad slackpad added the theme/api Relating to the HTTP API interface label May 25, 2017
@bgilner
Copy link

bgilner commented Jun 14, 2017

I would find this useful as it is very hacky to implement a workaround. My micro-service not only has to know where the consul url is but now a path to the executable just to run a consul watch command that invokes powershell that invokes a webrequest that my controller uses to reload settings.

More painful since consul.exe must either be added to the path of each environment or relative to each consuming service.

@slackpad slackpad added this to the Unplanned milestone Jan 5, 2018
@slackpad slackpad removed the post-0.9 label Jan 5, 2018
@shcherbachev
Copy link

I vote for this feature as well.

We have an HTTP endpoint listening for "watch" events implemented as a part of our service. Having to register a watch via agent's configuration files isn't that handy since services tend to go up and down.

It would be nice to have either:

  1. "watches" section available as a part of service definition
  2. http api so we can register/deregister a watch dynamically at service startup/shutdown
  3. get the HTTP handler type improved so it can point to a Consul's service (hostname name and port). I can use Consul's DNS record in http action right now, but not the port number. Plus we need an option to notify all service instances, not just one.

Personally, I prefer #3. Imagenary config with two new attributes: service, broadcast, and path starting with /:

"handler_type": "http",
"http_handler_config": {
    "service":"my_service",
    "broadcast":"true",
    "path":"/watch",
    "method": "POST"
}

@boostbob
Copy link

cool feature, plz think about this.

@bdhayakar
Copy link

This feature will be helpful.

@HuangDayu
Copy link

So, does version 1.8.5 support this new feature?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
theme/api Relating to the HTTP API interface type/enhancement Proposed improvement or new feature
Projects
None yet
Development

No branches or pull requests

7 participants