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

Jump to bottom

LazyLoader breaks Jinja shorthand for referencing execution modules #22444

Closed
whiteinge opened this issue Apr 7, 2015 · 1 comment
Closed

LazyLoader breaks Jinja shorthand for referencing execution modules #22444

whiteinge opened this issue Apr 7, 2015 · 1 comment
Labels
Bug broken, incorrect, or confusing behavior Needs-Testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases P3 Priority 3 Regression The issue is a bug that breaks functionality known to work in previous releases. severity-critical top severity, seen by most users, serious issues severity-low 4th level, cosemtic problems, work around exists
Milestone
Lithium

Comments

@whiteinge
Copy link
Contributor

Code:

{% set is_true = salt.modules.test.ping() %}

Error:

Jinja variable 'salt.loader.LazyLoader object' has no attribute 'test'
@whiteinge whiteinge added Bug broken, incorrect, or confusing behavior severity-low 4th level, cosemtic problems, work around exists Regression The issue is a bug that breaks functionality known to work in previous releases. Needs-Testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases P3 Priority 3 labels Apr 7, 2015
@whiteinge whiteinge added this to the Lithium milestone Apr 7, 2015
@whiteinge whiteinge mentioned this issue Apr 8, 2015
Closed
@whiteinge whiteinge added the severity-critical top severity, seen by most users, serious issues label Apr 10, 2015
@whiteinge
Copy link
Contributor Author

Fixed in #22723.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior Needs-Testcase PR needs test cases written, or the issue is about a bug/feature that needs test cases P3 Priority 3 Regression The issue is a bug that breaks functionality known to work in previous releases. severity-critical top severity, seen by most users, serious issues severity-low 4th level, cosemtic problems, work around exists
Projects
None yet
Milestone
Lithium
Development

No branches or pull requests
1 participant
@whiteinge