Closed miabbott closed 6 years ago
This PR removes the use include: across the roles to use include_tasks or include_role
include:
include_tasks
include_role
The most notable change is moving the set_is_atomic role from the common directory to the roles directory.
set_is_atomic
common
roles
The redhat_subscription role looks like it changed, but it really just got re-indented.
redhat_subscription
This partially addresses #336. I'm going to work on updating the tests in a separate PR.
Probably need requirements.txt updated for this too
I'm just going to close this in favor of all the changes in #373
This PR removes the use
include:
across the roles to useinclude_tasks
orinclude_role
The most notable change is moving the
set_is_atomic
role from thecommon
directory to theroles
directory.The
redhat_subscription
role looks like it changed, but it really just got re-indented.