suite.merge.config_merge: Skip empty postmerge

Signed-off-by: Zack Cerza <zack@redhat.com>

diff --git a/teuthology/suite/merge.py b/teuthology/suite/merge.py
index 647fe6e1d5c7552ebf11d78443052bc3b2a604f1..b46a0434b165f25c7dc3447577aaac8968be416f 100644 (file)
--- a/teuthology/suite/merge.py
+++ b/teuthology/suite/merge.py
@@ -154,17 +154,18 @@ def config_merge(configs, suite_name=None, **kwargs):
             deep_merge(yaml_complete_obj, yaml_fragment_obj)
 
         postmerge = yaml_complete_obj.get('teuthology', {}).get('postmerge', [])
-        postmerge = "\n".join(postmerge)
-        log.debug("postmerge script running:\n%s", postmerge)
-        env, script = new_script(postmerge, log, deep_merge, yaml.safe_load)
-        env['base_frag_paths'] = [strip_fragment_path(x) for x in paths]
-        env['description'] = desc
-        env['frag_paths'] = paths
-        env['suite_name'] = suite_name
-        env['yaml'] = yaml_complete_obj
-        for k,v in kwargs.items():
-            env[k] = v
-        if not script():
-            log.debug("skipping config %s due to postmerge filter", desc)
-            continue
+        if postmerge:
+            postmerge = "\n".join(postmerge)
+            log.debug("postmerge script running:\n%s", postmerge)
+            env, script = new_script(postmerge, log, deep_merge, yaml.safe_load)
+            env['base_frag_paths'] = [strip_fragment_path(x) for x in paths]
+            env['description'] = desc
+            env['frag_paths'] = paths
+            env['suite_name'] = suite_name
+            env['yaml'] = yaml_complete_obj
+            for k, v in kwargs.items():
+                env[k] = v
+            if not script():
+                log.debug("skipping config %s due to postmerge filter", desc)
+                continue
         yield desc, paths, yaml_complete_obj