commit 8b7715bf527f04c1ec028277752f3abbd118bf4c
author Anton Samoylov <asamoylov@mirantis.com> Mon Feb 18 16:01:11 2019 +0400
committer Anton Samoylov <asamoylov@mirantis.com> Mon Feb 18 16:09:38 2019 +0400
tree 6816810374d12a44cc02d725579dfc0c0546d2d8
parent 605b7b67cdfdbd2ab88a97380a96eb69b21c5074

Fix default behavior of zookeeper backup state:
- fix crontab schedule;
- do not set zookeeper backup via cron by default as it should be
  explicitely configured and controlled by cluster operators via cluster model.

Change-Id: I8c574aa7108d74bea2c74cb8d170cb470fa48b3c
Related-PROD: PROD-27603

zookeeper/backup.sls

diff --git a/zookeeper/backup.sls b/zookeeper/backup.sls
index 39ddbc0..bbb9169 100644
--- a/zookeeper/backup.sls
+++ b/zookeeper/backup.sls
@@ -54,6 +54,7 @@
   - hour: 2
 {%- endif %}
 {%- else %}
+  - minute: 0
   - hour: 2
 {%- endif %}
   - require:

zookeeper/map.jinja

diff --git a/zookeeper/map.jinja b/zookeeper/map.jinja
index 04eafe4..8bc8bff 100644
--- a/zookeeper/map.jinja
+++ b/zookeeper/map.jinja
@@ -24,13 +24,13 @@
     pkgs:
     - rsync
     backup_dir: '/var/backups/zookeeper'
-    cron: True
+    cron: False
     restrict_clients: True
   RedHat:
     pkgs:
     - rsync
     backup_dir: '/var/backups/zookeeper'
-    cron: True
+    cron: False
     restrict_clients: True
 
 {%- endload %}