openstack/bifrost
Code Issues Proposed changes

Dynamic inventory: do not set network_interface

Browse Source 
The same variable is used in Bifrost for a different purpose.

Change-Id: I3c9023a428314a83c0627496a7c488814efd837b
This commit is contained in:
Dmitry Tantsur 2022-02-16 16:36:19 +01:00
parent 72b267e90d
commit 5821a662f5
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
bifrost/inventory.py
View File

@ -316,6 +316,11 @@ def main():
if len(groups[group]['hosts']) == 0: if len(groups[group]['hosts']) == 0:
del groups[group] del groups[group]
# FIXME(dtantsur): there is a conflict between the Bifrost's and the node's
# network_interface. Drop the node's one for now.
for host, items in hostvars.items():
items.pop('network_interface', None)
# General Data Conversion # General Data Conversion
inventory = {'_meta': {'hostvars': hostvars}} inventory = {'_meta': {'hostvars': hostvars}}

6
releasenotes/notes/network-interface-ad1396813b83bf38.yaml Normal file
View File

@ -0,0 +1,6 @@
---
fixes:
- |
Fixes the Bifrost inventory plugin to not set the ``network_interface``
variable since it conflicts with the Bifrost's variable with a different
meaning.