Only apply the unroll wrapper once (#72003) (#72017)

Co-authored-by: Matt Martz <matt@sivel.net> Fixes #71920 (cherry picked from commit 4197666)
ansible · Oct 21, 2020 · d53d247 · d53d247
1 parent cd89aad
commit d53d247
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/changelogs/fragments/71920-fix-templating-recursion-error.yml b/changelogs/fragments/71920-fix-templating-recursion-error.yml
@@ -0,0 +1,2 @@
+bugfixes:
+  - Fix ``RecursionError`` when templating large vars structures (https://github.com/ansible/ansible/issues/71920)
diff --git a/lib/ansible/template/__init__.py b/lib/ansible/template/__init__.py
@@ -257,6 +257,7 @@ def wrapper(*args, **kwargs):
             return list(ret)
         return ret
 
+    wrapper.__UNROLLED__ = True
     return _update_wrapper(wrapper, func)
 
 
@@ -1025,7 +1026,8 @@ def do_template(self, data, preserve_trailing_newlines=True, escape_backslashes=
             # Adds Ansible custom filters and tests
             myenv.filters.update(self._get_filters())
             for k in myenv.filters:
-                myenv.filters[k] = _unroll_iterator(myenv.filters[k])
+                if not getattr(myenv.filters[k], '__UNROLLED__', False):
+                    myenv.filters[k] = _unroll_iterator(myenv.filters[k])
             myenv.tests.update(self._get_tests())
 
             if escape_backslashes: