Merge pull request #960 from kwoodson/httptestfix

Fixing the step parameters with a "no" when not passed with that key

1 files changed, 9 insertions, 1 deletions

diff --git a/roles/lib_zabbix/library/zbx_httptest.py b/roles/lib_zabbix/library/zbx_httptest.py
index 96733b3d1..6b28117ad 100644
--- a/roles/lib_zabbix/library/zbx_httptest.py
+++ b/roles/lib_zabbix/library/zbx_httptest.py
@@ -131,6 +131,14 @@ def steps_equal(zab_steps, user_steps):
 
     return True
 
+def process_steps(steps):
+    '''Preprocess the step parameters'''
+    for idx, step in enumerate(steps):
+        if not step.has_key('no'):
+            step['no'] = idx + 1
+
+    return steps
+
 # The branches are needed for CRUD and error handling
 # pylint: disable=too-many-branches
 def main():
@@ -218,7 +226,7 @@ def main():
                   'hostid': hostid,
                   'agent': module.params['agent'],
                   'retries': module.params['retries'],
-                  'steps': module.params['steps'],
+                  'steps': process_steps(module.params['steps']),
                   'applicationid': get_app_id(zapi, module.params['application']),
                   'delay': module.params['interval'],
                   'verify_host': get_verify_host(module.params['verify_host']),