sous-chefs / consul

Development repository for the consul cookbook

Home Page:https://supermarket.chef.io/cookbooks/consul

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Restart system.d service during consul version update

123BLiN opened this issue · comments

Maybe I'm missing something, but I can not find this in the cookbook.
Is it expected that I need to manually restart service on linux on every node (centos 7) during version upgrade?
75f76d6
Is it possible to make this as an attribute to be able to override it in the wrapper?
What were the reasons for such decision?
Btw on windows upgrade works with restart.

Hi @123BLiN . Yes, you got it right: #381 (comment)

Oh, thank you!

The problem as I see it, is that we are not able to override this default behaviour for consul client nodes (not server nodes)

So have this setting as an attirbute may be much appreciated, so I will try to leave this ticket open.
I'm trying to override this in the resources("consul_service[consul]") but no luck so far :(

This is likely something we could easily make configurable using the poise_service_options resource. It looks like this is overridable using node attributes as well.

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

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.