You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
What do you think of moving the consul_directories.each block from _service.rb to the install_*.rb recipes? That way, consul::install_binary could be run in an initial setup step, monitored services could be added to that directory, and later on the consul service could be enabled and started in a later step without needing to fuss with creating node['consul']['config_dir'] for every service.
I'd be happy to create a PR if you'd like.
Thanks for this great cookbook regardless!
The text was updated successfully, but these errors were encountered:
@rstrox it's in _service right now because it's common configuration between the two different installation methods. This could easily be refactored into a _common recipe and then included in each of the installation recipes.
I'm having a bit of a hard time understanding the way that you're using the cookbook that the order in which these directories are created is affecting you. Could you try to explain a bit more? Thanks!
My plan was to install consul early alongside a "base" set of other recipes, populate config_dir with files in later recipes, and then cluster the nodes outside of chef after all the nodes converge.
Is there a recommended workflow for bootstrapping a cluster if one does not already exist, and joining an existing cluster using Chef's search facility or similar?
What do you think of moving the
consul_directories.each
block from_service.rb
to theinstall_*.rb
recipes? That way,consul::install_binary
could be run in an initial setup step, monitored services could be added to that directory, and later on the consul service could be enabled and started in a later step without needing to fuss with creatingnode['consul']['config_dir']
for every service.I'd be happy to create a PR if you'd like.
Thanks for this great cookbook regardless!
The text was updated successfully, but these errors were encountered: