Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

Allow when inside import_playbook #327

Merged
merged 1 commit into from
Jun 24, 2022

Conversation

apatard
Copy link
Contributor

@apatard apatard commented Jun 23, 2022

According to https://docs.ansible.com/ansible/latest/user_guide/playbooks_conditionals.html#conditionals-with-imports,
ansible allows to use conditional with import_playbook:

"You can apply conditions to import_playbook as well as to the other import_* statements."

The current schema does not allow it, so add support for it.

[ As side effect, update a bunch of markdown failure files to have the correct
message with when: ]

Fixes: #316
Signed-off-by: Arnaud Patard [email protected]

According to https://docs.ansible.com/ansible/latest/user_guide/playbooks_conditionals.html#conditionals-with-imports,
ansible allows to use conditional with import_playbook:

"You can apply conditions to import_playbook as well as to the other import_* statements."

The current schema does not allow it, so add support for it.

[ As side effect, update a bunch of markdown failure files to have the correct
message with when: ]

Fixes: #316
Signed-off-by: Arnaud Patard <[email protected]>
@apatard apatard requested review from ssbarnea and a team as code owners June 23, 2022 11:27
@ssbarnea ssbarnea added the bug Something isn't working label Jun 24, 2022
@ssbarnea ssbarnea changed the title f/ansible.json: Allow conditional with import_playbook. Allow when inside import_playbook Jun 24, 2022
@ssbarnea ssbarnea merged commit 302879b into ansible:main Jun 24, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistency between Ansible documentation and Schemas?
2 participants