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

Template role_name when statically including via include_role #31463

Merged
merged 1 commit into from
Oct 9, 2017

Conversation

jimi-c
Copy link
Member

@jimi-c jimi-c commented Oct 9, 2017

Also trims out code which was incorrectly setting is_static=True for
include_role (and include_task, possibly).

Fixes #30700

@jimi-c
Copy link
Member Author

jimi-c commented Oct 9, 2017

bot_skip

Also trims out code which was incorrectly setting is_static=True for
include_role.

Fixes ansible#30700
@jimi-c jimi-c force-pushed the issue_30700_role_include_var_name branch from 481f955 to 8bd8232 Compare October 9, 2017 14:20
@jimi-c jimi-c merged commit e609618 into ansible:devel Oct 9, 2017
@abadger abadger added this to In Progress in 2.4.x Blocker List Oct 9, 2017
@abadger abadger removed this from In Progress in 2.4.x Blocker List Oct 9, 2017
abadger pushed a commit that referenced this pull request Oct 9, 2017
Also trims out code which was incorrectly setting is_static=True for
include_role.

Fixes #30700
(cherry picked from commit e609618)
BondAnthony pushed a commit to BondAnthony/ansible that referenced this pull request Oct 14, 2017
…e#31463)

Also trims out code which was incorrectly setting is_static=True for
include_role.

Fixes ansible#30700
@ansible ansible locked and limited conversation to collaborators Apr 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

vars used in include_role or import_role are evaluated too early
1 participant