Commit 5b81f88c by Steve Smith

Cast the retrieved `retries` var to an int before incrementing as it may be in string form.

For example, the following method of calculating the value will result in a type error:

    appstatus_waitfor: 4  # Minutes
    appstatus_delay: 5 # seconds
    appstatus_retries: "{{ mins * 60 / delay }}"
parent 36a305ce
...@@ -691,7 +691,7 @@ class Runner(object): ...@@ -691,7 +691,7 @@ class Runner(object):
if not utils.check_conditional(cond, self.basedir, inject, fail_on_undefined=self.error_on_undefined_vars): if not utils.check_conditional(cond, self.basedir, inject, fail_on_undefined=self.error_on_undefined_vars):
retries = self.module_vars.get('retries') retries = self.module_vars.get('retries')
delay = self.module_vars.get('delay') delay = self.module_vars.get('delay')
for x in range(1, retries + 1): for x in range(1, int(retries) + 1):
# template the delay, cast to float and sleep # template the delay, cast to float and sleep
delay = template.template(self.basedir, delay, inject, expand_lists=False) delay = template.template(self.basedir, delay, inject, expand_lists=False)
delay = float(delay) delay = float(delay)
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment