Ensure Jinja2 template header overrides are used (#75306)

Fixes #75275
pull/75337/head
Martin Krizek 3 years ago committed by GitHub
parent e5a2fe4c41
commit 767b2f07b0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

@ -0,0 +1,2 @@
bugfixes:
- template - ensure Jinja2 overrides from template header are used (https://github.com/ansible/ansible/issues/75275)

@ -1079,15 +1079,21 @@ class Templar:
if fail_on_undefined is None: if fail_on_undefined is None:
fail_on_undefined = self._fail_on_undefined_errors fail_on_undefined = self._fail_on_undefined_errors
has_template_overrides = data.startswith(JINJA2_OVERRIDE)
try: try:
# allows template header overrides to change jinja2 options. # NOTE Creating an overlay that lives only inside do_template means that overrides are not applied
if overrides is None: # when templating nested variables in AnsibleJ2Vars where Templar.environment is used, not the overlay.
myenv = self.environment # This is historic behavior that is kept for backwards compatibility.
else: if overrides:
myenv = self.environment.overlay(overrides) myenv = self.environment.overlay(overrides)
elif has_template_overrides:
myenv = self.environment.overlay()
else:
myenv = self.environment
# Get jinja env overrides from template # Get jinja env overrides from template
if hasattr(data, 'startswith') and data.startswith(JINJA2_OVERRIDE): if has_template_overrides:
eol = data.find('\n') eol = data.find('\n')
line = data[len(JINJA2_OVERRIDE):eol] line = data[len(JINJA2_OVERRIDE):eol]
data = data[eol + 1:] data = data[eol + 1:]

@ -0,0 +1,5 @@
#jinja2:variable_start_string:'<<' , variable_end_string:'>>'
var_a: << var_a >>
var_b: << var_b >>
var_c: << var_c >>
var_d: << var_d >>

@ -0,0 +1,28 @@
- hosts: localhost
gather_facts: false
vars:
var_a: "value"
var_b: "{{ var_a }}"
var_c: "<< var_a >>"
tasks:
- set_fact:
var_d: "{{ var_a }}"
- block:
- template:
src: in_template_overrides.j2
dest: out.txt
- command: cat out.txt
register: out
- assert:
that:
- "'var_a: value' in out.stdout"
- "'var_b: value' in out.stdout"
- "'var_c: << var_a >>' in out.stdout"
- "'var_d: value' in out.stdout"
always:
- file:
path: out.txt
state: absent

@ -38,3 +38,5 @@ ansible-playbook 72262.yml -v "$@"
# ensure unsafe is preserved, even with extra newlines # ensure unsafe is preserved, even with extra newlines
ansible-playbook unsafe.yml -v "$@" ansible-playbook unsafe.yml -v "$@"
# ensure Jinja2 overrides from a template are used
ansible-playbook in_template_overrides.yml -v "$@"

Loading…
Cancel
Save