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

Filter out bootstrap and bootstrap_expect parameters in client configs #364

Closed gdavison closed 7 years ago

gdavison commented 7 years ago

Filter out some server-specific parameters from client config files. Consul 0.7 refuses to start when, for instance, bootstrap_expect is present.

When we set up our development environments, we set the parameter on the whole Chef environment, as the number of Consul servers seems to fit as a property of the environment. So when I tried updating one of the dev environments today, each client machine failed.

I haven't tested which other parameters will cause failure, if any

codecov-io commented 7 years ago

Current coverage is 71.92% (diff: 100%)

No coverage report found for master at 742a56a.

Powered by Codecov. Last update 742a56a...3820243

gdavison commented 7 years ago

Blocked by failing RSpec tests on master

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.