search

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

Allow specification of shell for the consul user #349

Closed jirkafajfr closed 8 years ago

jirkafajfr commented 8 years ago

Hi,

following two commits are changing shell of the consul user to /bin/nologin (or its variations)

this configuration prevents consul from executing handler in consul watch with following error:

2016/09/20 21:33:02 [ERR] agent: Failed to invoke watch handler 'XYZ': exit status 1

This patch allows wrapper cookbook to specify shell explicitly while retaining "security" for people not using consul watch functionality.

codecov-io commented 8 years ago

Current coverage is 60.45% (diff: 100%)

No coverage report found for master at a43dc10.

Powered by Codecov. Last update a43dc10...379ca3b

quulah commented 8 years ago

There's also the old issue #271. So the scripts checks seem to be broken too without the user's shell.

lock[bot] commented 4 years ago

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.