windmill/ansible-role-zookeeper
Code Issues Proposed changes

Install zoo.cfg into the proper directory

Browse Source 
It looks like there is a bug where zookeeper_file_zoo_conf_dest isn't
setup properly on debubuntu. Support both now.

Change-Id: I9ecd3171005f162dcae3d03f0178a40b9626e104
Depends-On: https://review.openstack.org/622292
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
This commit is contained in:
Paul Belanger 2018-12-03 11:55:53 -05:00
parent 4dd7a241b1
commit 16b495b358
4 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
defaults/main.yaml
View File

@ -19,11 +19,9 @@ zookeeper_task_manager:
- service - service
# tasks/config.yaml # tasks/config.yaml
zookeeper_file_zoo_conf_dest: /etc/zookeeper/zoo.cfg
zookeeper_file_zoo_conf_group: zookeeper zookeeper_file_zoo_conf_group: zookeeper
zookeeper_file_zoo_conf_mode: 0644 zookeeper_file_zoo_conf_mode: 0644
zookeeper_file_zoo_conf_owner: zookeeper zookeeper_file_zoo_conf_owner: zookeeper
zookeeper_file_zoo_conf_src: etc/zookeeper/zoo.cfg
# tasks/install.yaml # tasks/install.yaml
zookeeper_package_state: installed zookeeper_package_state: installed

0
templates/etc/zookeeper/zoo.cfg → templates/etc/zookeeper/conf/zoo.cfg
View File

3
vars/Debian.yaml
View File

@ -26,3 +26,6 @@
# under the License. # under the License.
--- ---
__zookeeper_package_name: zookeeperd __zookeeper_package_name: zookeeperd
zookeeper_file_zoo_conf_dest: /etc/zookeeper/conf/zoo.cfg
zookeeper_file_zoo_conf_src: etc/zookeeper/conf/zoo.cfg

4
vars/RedHat.yaml
View File

@ -13,3 +13,7 @@
# under the License. # under the License.
--- ---
__zookeeper_package_name: zookeeper __zookeeper_package_name: zookeeper
zookeeper_file_zoo_conf_dest: /etc/zookeeper/zoo.cfg
# NOTE(pabelanger): Avoid creating a symlink, and point to Debian version.
zookeeper_file_zoo_conf_src: etc/zookeeper/conf/zoo.cfg