-
Notifications
You must be signed in to change notification settings - Fork 3
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
v2 templating config #14
Comments
Thanks for raising this issue. Similar issues to what you're seeing have been discussed in #12. Are you able to try using the |
@dbrennand I did indeed give that a shot but the generated config file was pretty rough and it wasn't clear how to solve the formatting.
template for reference:
|
Hi @PleaseStopAsking - I've pushed the following commit 2027064 Are you able to test this version of the role on branch issue/#12? |
@dbrennand This change does appear to solve the problem. Below is the usage just for reference.
|
That's great news. Thank you for testing @PleaseStopAsking 🙂 I think we can move forward with this implementation because the user can either override the template themselves, as you've done here, or they can continue using |
I am in the process of updating my usage of this role and with the recent switch from string to yaml for the config, I am running into an issue and have been unable to workaround it myself so far.
I am leveraging Jinja2 loop to handle some locations and backends but with the new YAML change, those loops are causing issues.
I am hoping this is something simple I am missing.
Example variable:
Current autorestic config that worked before the yaml change
The text was updated successfully, but these errors were encountered: