You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
With the new-style of playbook, - include:ing other sub--playbooks it seems extraneous to run fact checking on every included playbook run (for example myne has 12 - include:s) and instead using gather_facts: False in every playbook except the first one.
Might be worth updating configuration.yml with a comment about why it's disabled to explicate that, but again don't want to start mucking things up until I've heard from @rricard.
The text was updated successfully, but these errors were encountered:
gather_facts: False is totally ok to me, we should do that as long as the user doesn't want to deploy on other machines while doing service discovery So, yes we should default it in the example playbook BUT we should let a note saying thatit shouldn't be used in some cases.
With the new-style of playbook,
- include:
ing other sub--playbooks it seems extraneous to run fact checking on every included playbook run (for example myne has 12- include:
s) and instead usinggather_facts: False
in every playbook except the first one.Might be worth updating
configuration.yml
with a comment about why it's disabled to explicate that, but again don't want to start mucking things up until I've heard from @rricard.The text was updated successfully, but these errors were encountered: