openstack/ironic
Code Issues Proposed changes

Merge "Clarify image_source with BFV"

Browse Source
This commit is contained in:
Zuul
2018-05-04 16:03:02 +00:00
committed by Gerrit Code Review
parent b150125a13 863aa34145
commit 94b10d7ab3
2 changed files with 9 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
doc/source/contributor/ironic-boot-from-volume.rst
View File

@@ -124,3 +124,9 @@ volume with tempest in the environment::
cd /opt/stack/tempest cd /opt/stack/tempest
tox -e all-plugin -- ironic_tempest_plugin.tests.scenario.test_baremetal_boot_from_volume tox -e all-plugin -- ironic_tempest_plugin.tests.scenario.test_baremetal_boot_from_volume
Please note that the storage interface will only indicate errors based upon
the state of the node and the configuration present. As such a node does not
exclusively have to boot via a remote volume, and as such `validate` actions
upon nodes may be slightly misleading. If an appropriate `volume target` is
defined, no error should be returned for the boot interface.

4
ironic/drivers/modules/deploy_utils.py
View File

@@ -1050,7 +1050,9 @@ def get_image_instance_info(node):
info['ramdisk'] = node.instance_info.get('ramdisk') info['ramdisk'] = node.instance_info.get('ramdisk')
error_msg = (_("Cannot validate image information for node %s because one " error_msg = (_("Cannot validate image information for node %s because one "
"or more parameters are missing from its instance_info") "or more parameters are missing from its instance_info and "
"insufficent information is present to boot from a remote "
"volume")
% node.uuid) % node.uuid)
check_for_missing_params(info, error_msg) check_for_missing_params(info, error_msg)