openstack
/
deb-python-os-cloud-config
Code Issues Proposed changes

Replace ucs_hostname with ucs_address 

The ucs_hostname option never existed in the UCS driver [0], it
was a mistake in the driver's documentation which was later on
fixed by [1].

[0] https://review.openstack.org/#/c/159734/
[1] https://review.openstack.org/#/c/232834/

Change-Id: I83aace3ec70ab211d4c4167fc151c38ac8e81600
This commit is contained in:
Lucas Alvares Gomes 2016-09-15 13:39:44 +01:00
parent e1dca0031e
commit 71259bbc09
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
os_cloud_config/nodes.py
View File

@ -86,7 +86,7 @@ def _fake_pxe_driver_info(node):
def _pxe_ucs_driver_info(node):
driver_info = {"ucs_hostname": node["pm_addr"],
driver_info = {"ucs_address": node["pm_addr"],
"ucs_username": node["pm_user"],
"ucs_password": node["pm_password"],
"ucs_service_profile": node["pm_service_profile"]}

2
os_cloud_config/tests/test_nodes.py
View File

@ -129,7 +129,7 @@ class NodesTest(base.TestCase):
node = self._get_node()
node["pm_type"] = "pxe_ucs"
node["pm_service_profile"] = "foo_profile"
expected = {"ucs_hostname": "foo.bar",
expected = {"ucs_address": "foo.bar",
"ucs_username": "test",
"ucs_password": "random",
"ucs_service_profile": "foo_profile"}