search

ansible-collections / ansible-consul

:satellite: Ansible role for Hashicorp Consul clusters
https://galaxy.ansible.com/ansible-community/consul/
BSD 2-Clause "Simplified" License
457 stars 317 forks source link

Add main configuration files to managed_files list #464

Closed wawa19933 closed 2 years ago

wawa19933 commented 2 years ago

The difference between consul_config_path and consul_configd_path was not clear to me at a first glance. That's why I have set them to the same value. This led me to the state, when all config files are removed during deployment. Because in services.yml all files within consul_configd_path which are not present in managed_files list are removed.

My change will save configuration files from accidential remove by other task in this role. It seems rather sane to add thses files to managed_files list as well.

bbaassssiiee commented 2 years ago

Tests failed, please check if still valid.