Fix designate-manage pool update retries
This patch fixes bug when result e variable wasn't set by default/
As result default value for it was set by first attempt, and in case
of failure command always exited with fail status, even second
attempt succeded.
Change-Id: I412b0bdbdd90bc2f0fed31dd49cf83f7e80206c9
Related-Prod: PROD-18908
diff --git a/designate/server.sls b/designate/server.sls
index c8b2eb8..d860909 100644
--- a/designate/server.sls
+++ b/designate/server.sls
@@ -85,7 +85,8 @@
{%- if not grains.get('noservices', False) %}
designate_pool_update:
cmd.run:
- - name: for i in {1..3}; do designate-manage pool update && break || e=$? && sleep 10; done; (exit $e)
+ - name: "attempts=3; for i in $(seq 1 $attempts); do if designate-manage pool update; then exit 0; else if [[ $i -ge $attempts ]]; then exit 1; else sleep 10; fi; fi; done"
+ - shell: /bin/bash
- require:
- service: designate_server_services
- onchanges: