openstack/bifrost
Code Issues Proposed changes

Merge "Force local connection for localhost"

Browse Source
This commit is contained in:
Zuul
2020-10-22 02:37:20 +00:00
committed by Gerrit Code Review
parent 0545bf607c a6c56c6647
commit a77df17a55
3 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
bifrost/inventory.py
View File

@@ -164,7 +164,7 @@ def _parse_config():
def _prepare_inventory():
hostvars = {}
hostvars = {"127.0.0.1": {"ansible_connection": "local"}}
groups = {}
groups.update({'baremetal': {'hosts': []}})
groups.update({'localhost': {'hosts': ["127.0.0.1"]}})

2
bifrost/tests/functional/test_inventory_functional.py
View File

@@ -53,6 +53,7 @@ class TestBifrostInventoryFunctional(base.TestCase):
"host_groups": ["baremetal", "nova"]}}""".replace('\n', '')
(groups, hostvars) = utils.bifrost_data_conversion(
yaml.safe_dump(json.loads(str(expected_hostvars))))
del hostvars['127.0.0.1']
self.assertDictEqual(json.loads(str(expected_hostvars)), hostvars)
def test_minimal_json(self):
@@ -66,4 +67,5 @@ ipmi_password":"ADMIN"}},"driver":"ipmi"}}""".replace('\n', '')
ipmi_password":"ADMIN"}},"driver":"ipmi","addressing_mode":
"dhcp","host_groups": ["baremetal"]}}""".replace('\n', '')
(groups, hostvars) = utils.bifrost_data_conversion(input_json)
del hostvars['127.0.0.1']
self.assertDictEqual(json.loads(str(expected_json)), hostvars)

3
bifrost/tests/unit/test_inventory.py
View File

@@ -33,7 +33,8 @@ class TestBifrostInventoryUnit(base.TestCase):
(groups, hostvars) = inventory._prepare_inventory()
self.assertIn("baremetal", groups)
self.assertIn("localhost", groups)
self.assertDictEqual(hostvars, {})
self.assertDictEqual(
hostvars, {'127.0.0.1': {'ansible_connection': 'local'}})
localhost_value = dict(hosts=["127.0.0.1"])
self.assertDictEqual(localhost_value, groups['localhost'])