Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
A
ansible
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
OpenEdx
ansible
Commits
845784b5
Commit
845784b5
authored
Jul 30, 2015
by
Abhijit Menon-Sen
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Remove outdated FIXMEs; all use combine_vars already
parent
efcad3aa
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
0 additions
and
8 deletions
+0
-8
lib/ansible/inventory/__init__.py
+0
-8
No files found.
lib/ansible/inventory/__init__.py
View file @
845784b5
...
...
@@ -151,12 +151,10 @@ class Inventory(object):
# management will be done in VariableManager
# get group vars from group_vars/ files and vars plugins
for
group
in
self
.
groups
:
# FIXME: combine_vars
group
.
vars
=
combine_vars
(
group
.
vars
,
self
.
get_group_variables
(
group
.
name
))
# get host vars from host_vars/ files and vars plugins
for
host
in
self
.
get_hosts
():
# FIXME: combine_vars
host
.
vars
=
combine_vars
(
host
.
vars
,
self
.
get_host_variables
(
host
.
name
))
...
...
@@ -441,11 +439,9 @@ class Inventory(object):
vars_results
=
[
plugin
.
get_group_vars
(
group
,
vault_password
=
vault_password
)
for
plugin
in
self
.
_vars_plugins
if
hasattr
(
plugin
,
'get_group_vars'
)]
for
updated
in
vars_results
:
if
updated
is
not
None
:
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
updated
)
# Read group_vars/ files
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
self
.
get_group_vars
(
group
))
return
vars
...
...
@@ -475,25 +471,21 @@ class Inventory(object):
vars_results
=
[
plugin
.
run
(
host
,
vault_password
=
vault_password
)
for
plugin
in
self
.
_vars_plugins
if
hasattr
(
plugin
,
'run'
)]
for
updated
in
vars_results
:
if
updated
is
not
None
:
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
updated
)
# plugin.get_host_vars retrieves just vars for specific host
vars_results
=
[
plugin
.
get_host_vars
(
host
,
vault_password
=
vault_password
)
for
plugin
in
self
.
_vars_plugins
if
hasattr
(
plugin
,
'get_host_vars'
)]
for
updated
in
vars_results
:
if
updated
is
not
None
:
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
updated
)
# still need to check InventoryParser per host vars
# which actually means InventoryScript per host,
# which is not performant
if
self
.
parser
is
not
None
:
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
self
.
parser
.
get_host_variables
(
host
))
# Read host_vars/ files
# FIXME: combine_vars
vars
=
combine_vars
(
vars
,
self
.
get_host_vars
(
host
))
return
vars
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment