mattfinlayson / ansible-consul

Ansible role for consul
Apache License 2.0
233 stars 181 forks source link

consul config file does not notify handler #15

Open bryanlarsen opened 9 years ago

bryanlarsen commented 9 years ago

If the consul config file changes, it should notify a handler to restart consul.

robbiet480 commented 9 years ago

:+1: on this. I guess i'll just manually run the handler. It may be good to add a handler to reload consul instead of restarting it, if it's only for a configuration change

MrMMorris commented 9 years ago

I would definitely try to avoid restarting automatically especially with Consul Servers. Reload makes sense though

trumant commented 9 years ago

The one problem is that we'd need to be really smart in the handler to determine whether reload or restart is needed given this: https://consul.io/docs/agent/options.html#reloadable-configuration