Restart service instead reload as only few configurations are reloadble.

[vijaybandari]

Reload works only for few set of configurations as mentioned here.
https://www.consul.io/docs/agent/options.html#reloadable-configuration

https://github.com/johnbellone/consul-cookbook/blob/master/recipes/default.rb#L45
It should be restart instead of reload.

[legal90]

Hi @vijaybandari
That was done intentionally by 80f4a1c and #309
Please, refer to the discussion: #288

TL;DR: Automatic restart of Consul servers could cause a cluster downtime when you run chef-client simultaneously on your nodes. Also, if Consul version has been changed (upgrade scenario), then Consul servers should be restarted manually, according to the upgrade manual.

[vijaybandari]

Thanks @legal90 , now it makes sense.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.