sous-chefs / consul

Development repository for the consul cookbook
https://supermarket.chef.io/cookbooks/consul
Apache License 2.0
192 stars 244 forks source link

acl_master_token must be set on non-server agents in order to reload windows service #487

Open brianbaker opened 6 years ago

brianbaker commented 6 years ago

The config option acl_master_token must be set on non-server agents in order to reload the consul service on windows. Per the Consul ACL docs though:

Non-server agents do not need to have the acl_master_token configured; it is not used by agents in any way.

For windows installations on non-server machines the consul reload fails. As a work-around, we can populate default['consul']['config']['acl_master_token'] with any ACL token that has write access to the agent policy. Is this the appropriate solution or is this a bug?

damacus commented 5 years ago

Hi there!

Thanks for taking the time to submit an issue.

We would happily accept a pull request to resolve this issue. If you need help or guidance on submitting, please don't hesitate to ask!

If you need help, or just wish to talk through issue please join the Chef Community Slack and look for the #sous-chefs channel.

Thanks, Sous-Chefs

github-actions[bot] commented 4 years ago

Marking stale due to inactivity. Remove stale label or comment or this will be closed in 7 days. Alternatively drop by the #sous-chefs channel on the Chef Community Slack and we'll be happy to help! Thanks, Sous-Chefs.