Setting consul_group fails deployment

[tommyalatalo]

Setting the consul_group variable to anything but "bin" fails.

This seems to be because the "consul_manage_group" variable is never set to true regardless of the value of consul_group. If this is intended behavior it doesn't say so in the docs, but I'm assuming it's not.

[jadams]

In the same vein since "consul_manage_group" is default to false, in some deployments root's primary group gets changed to "bin" (gid 2) which really messed up some things. I would recommend putting something in the readme or defaulting that to true.

[brianshumate]

Thanks for the heads up; I've changed the default value of consul_manage_group to true and pushed that to master so that it will be the default in the next release version.

[bbaassssiiee]

Please note that there is a history of issues related to the 'bin'group, and consider changing the value of the default to

consul_group: 'root'

What is the bin user for? https://unix.stackexchange.com/questions/485505/what-is-the-bin-user-for/485506#485506