Move searching for roles in the cur basedir to last
Searching the DEFAULT_ROLES_PATH and the roles basedir should come
before this, and it has been a long standing oversight.
Fixes #17882
(cherry picked from commit 0a86ddc251
)
This commit is contained in:
parent
dea2cabe94
commit
e4efe0b2f1
1 changed files with 6 additions and 2 deletions
|
@ -138,18 +138,22 @@ class RoleDefinition(Base, Become, Conditional, Taggable):
|
|||
# we always start the search for roles in the base directory of the playbook
|
||||
role_search_paths = [
|
||||
os.path.join(self._loader.get_basedir(), u'roles'),
|
||||
self._loader.get_basedir(),
|
||||
]
|
||||
|
||||
# also search in the configured roles path
|
||||
if C.DEFAULT_ROLES_PATH:
|
||||
role_search_paths.extend(C.DEFAULT_ROLES_PATH)
|
||||
|
||||
# finally, append the roles basedir, if it was set, so we can
|
||||
# next, append the roles basedir, if it was set, so we can
|
||||
# search relative to that directory for dependent roles
|
||||
if self._role_basedir:
|
||||
role_search_paths.append(self._role_basedir)
|
||||
|
||||
# finally as a last resort we look in the current basedir as set
|
||||
# in the loader (which should be the playbook dir itself) but without
|
||||
# the roles/ dir appended
|
||||
role_search_paths.append(self._loader.get_basedir())
|
||||
|
||||
# create a templar class to template the dependency names, in
|
||||
# case they contain variables
|
||||
if self._variable_manager is not None:
|
||||
|
|
Loading…
Reference in a new issue