diff options
| author | Brian Coca <brian.coca+git@gmail.com> | 2015-12-09 07:21:00 -0800 |
|---|---|---|
| committer | Brian Coca <brian.coca+git@gmail.com> | 2015-12-09 07:29:36 -0800 |
| commit | 87969868d42cd8aba1c65c8207d059d73407373b (patch) | |
| tree | 45c83b19f6605d38caebe03e5d374b59c85b399f /lib/ansible/playbook/playbook_include.py | |
| parent | 0719eb3e2d798c6f80223e37dd77bc0ac41c537d (diff) | |
| download | ansible-87969868d42cd8aba1c65c8207d059d73407373b.tar.gz | |
avoid persistent containers in attribute defaults
moved from the field attribute declaration and created a placeholder
which then is resolved in the field attribute class.
this is to avoid unwanted persistent of the defaults across objects which introduces
stealth bugs when multiple objects of the same kind are used in succession while
not overriding the default values.
Diffstat (limited to 'lib/ansible/playbook/playbook_include.py')
| -rw-r--r-- | lib/ansible/playbook/playbook_include.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/ansible/playbook/playbook_include.py b/lib/ansible/playbook/playbook_include.py index d9af2ba523..52081c4153 100644 --- a/lib/ansible/playbook/playbook_include.py +++ b/lib/ansible/playbook/playbook_include.py @@ -35,7 +35,7 @@ class PlaybookInclude(Base, Conditional, Taggable): _name = FieldAttribute(isa='string') _include = FieldAttribute(isa='string') - _vars = FieldAttribute(isa='dict', default=dict()) + _vars = FieldAttribute(isa='dict', default='_ansible_container') @staticmethod def load(data, basedir, variable_manager=None, loader=None): |