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.

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

salt.formulas fails on highstate if not data #131

Closed
txomon opened this issue May 8, 2015 · 3 comments · Fixed by #135
Closed

salt.formulas fails on highstate if not data #131

txomon opened this issue May 8, 2015 · 3 comments · Fixed by #135

Comments

@txomon
Copy link
Contributor

txomon commented May 8, 2015

I removed the salt_formulas: thing from my pillar, but forgot the salt/top.sls with the salt.formulas there and it started to fail.

[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/master.sls'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/map.jinja'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/defaults.yaml'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/minion.sls'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/map.jinja'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/defaults.yaml'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/formulas.sls'
[INFO    ] Fetching file from saltenv 'base', ** skipped ** latest already in cache 'salt://salt/formulas.jinja'
[CRITICAL] Rendering SLS 'base:salt.formulas' failed: Jinja variable 'str object' has no attribute 'iteritems'
local:
    Data failed to compile:
----------
    Rendering SLS 'base:salt.formulas' failed: Jinja variable 'str object' has no attribute 'iteritems'
@aboe76
Copy link
Member

aboe76 commented May 8, 2015

@txomon that's strange it should read the file formulas.jinja with the defaults.

@txomon
Copy link
Contributor Author

txomon commented May 8, 2015

Yes, I am not saying it should work, just that it should do nothing =)

I have tried to fix it but I've been unable to spot where, so I just filled the issue

@aboe76
Copy link
Member

aboe76 commented May 17, 2015

@txomon I have found the issue and fixed it PR #135

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants