Merge pull request #48 from ernestas-poskus/features/do_not_overwrite…

…_zoo_myid_unless_stated Do not overwrite Zookeeper myid unless stated
AnsibleShipyard · Sep 20, 2016 · 51df52c · 51df52c
2 parents 8829ac4 + d82afa4
commit 51df52c
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -61,6 +61,9 @@ zookeeper_hosts:
 
 # Dict of ENV settings to be written into the (optional) conf/zookeeper-env.sh
 zookeeper_env: {}
+
+# Controls Zookeeper myid generation
+zookeeper_force_myid: yes
 ```
 
 Example Playbook

diff --git a/defaults/main.yml b/defaults/main.yml
@@ -30,3 +30,6 @@ zookeeper_hosts:
 
 # Dict of ENV settings to be written into the (optional) conf/zookeeper-env.sh
 zookeeper_env: {}
+
+# Controls Zookeeper myid generation
+zookeeper_force_myid: yes
diff --git a/tasks/Debian.yml b/tasks/Debian.yml
@@ -11,7 +11,7 @@
     - zookeeperd
 
 - name: Overwrite myid file.
-  template: src=myid.j2 dest=/etc/zookeeper/conf/myid
+  template: src=myid.j2 dest=/etc/zookeeper/conf/myid force="{{ zookeeper_force_myid }}"
   tags: deploy
   notify:
     - Restart zookeeper

diff --git a/tasks/tarball.yml b/tasks/tarball.yml
@@ -28,7 +28,7 @@
     - "{{log_dir}}"
 
 - name: Write myid file.
-  template: src=myid.j2 dest={{data_dir}}/myid owner=zookeeper group=zookeeper
+  template: src=myid.j2 dest={{data_dir}}/myid owner=zookeeper group=zookeeper force="{{ zookeeper_force_myid }}"
   tags: deploy
   notify:
     - Restart zookeeper