Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix: Set loader on local copy of jinja environment #5

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Bugfix: Set loader on local copy of jinja environment #5

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
c2d5d7e
Add loader even if user specified custom jinja env. Seems like this g…
indy-independence Dec 11, 2020
66c4f67
template_string does not use path
indy-independence Dec 11, 2020
9503500
Add loader even if user specified custom jinja env. Seems like this g…
indy-independence Dec 11, 2020
94e42f3
template_string does not use path
indy-independence Dec 11, 2020
61f4893
Merge branch 'master' of github.com:indy-independence/nornir_jinja2
indy-independence Dec 11, 2020
f771467
Merge branch 'nornir-automation:master' into master
indy-independence May 19, 2021
2d19fb1
Change loader on local copy of env object
indy-independence May 19, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion nornir_jinja2/plugins/tasks/template_file.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
import copy
from typing import Any, Optional, Dict, Callable

from nornir.core.task import Result, Task
Expand Down Expand Up @@ -32,7 +33,7 @@ def template_file(
jinja_filters = jinja_filters or {}

if jinja_env:
env = jinja_env
env = copy.deepcopy(jinja_env)
env.loader = FileSystemLoader(path)
else:
env = Environment(
Expand Down