Skip to content

Commit

Permalink
docs: add alternative to include-role
Browse files Browse the repository at this point in the history
It feels appropriate to mention this alternative since
the author of the anti-pattern article seems to propose this approach too
(in the more recent comments of the article).
  • Loading branch information
Johannes Aalto committed Dec 15, 2021
1 parent 07e6bac commit 9f72011
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,8 @@ add this label to container_cmd_args: ```--label "io.containers.autoupdate=image
Never use `ansible.builtin.import_role` to execute this role if you intend to use it more
than once per playbook, or you will fall in
[this anti-pattern](https://medium.com/opsops/ansible-anti-pattern-import-role-task-with-task-level-vars-a9f5c752c9c3).
Alternatively, you could use the "roles" block on top level of play and specify the variables
[without "vars" block](https://medium.com/opsops/role-parameters-in-ansible-946386f32e77).

Dependencies
------------
Expand Down

0 comments on commit 9f72011

Please sign in to comment.