-
-
Notifications
You must be signed in to change notification settings - Fork 112
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.
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
Unable to deploy with latest version of Jinja2 #842
Comments
The place that it's failing is here, which could probably be re-written much simpler to let But we use b64decode in another place, and there'd probably be errors elsewhere too. |
Also looks similar to readthedocs/readthedocs.org#9038 |
jibees
added a commit
to jibees/ofn-install
that referenced
this issue
Nov 23, 2022
jibees
added a commit
to jibees/ofn-install
that referenced
this issue
Nov 23, 2022
jibees
added a commit
to jibees/ofn-install
that referenced
this issue
Nov 23, 2022
jibees
added a commit
to jibees/ofn-install
that referenced
this issue
Nov 23, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
We have an old version of Ansible which is no longer supported (v2.9 was EOL in May 2022). I think it is incompatible with the latest version of Jinja2, which cause issues for me when trying to deploy (see below).
Expected Behavior
The playbook works without errors or warnings
Actual Behaviour
And also an error, causing the deployment to fail: `template error while templating string: No filter named 'b64decode'
The place it fails is here: https://github.com/openfoodfoundation/ofn-install/blob/master/roles/deploy/tasks/deploy.yml#L44
Similar to: https://gitlab.com/dreamer-labs/libraries/jinja2-ansible-filters/-/merge_requests/15
Steps to Reproduce
Workaround
For now, you can simply use an older version of Jinja2:
Severity
bug-s5: we can live with it, only a few users impacted
Your Environment
In addition to above:
Possible Fix
I think we should upgrade to a later version of Ansible. Strangely it has incremented 4 major versions in the last two years, so it might be worth trying step by step:
The text was updated successfully, but these errors were encountered: