openstack/ironic
Code Issues Proposed changes

Clarify driver load error message

Browse Source 
The NoValidDefaultForInterface exception is a little misleading
in that if one doesn't have the base interface enabled, and they
attempt to enable a hardware type which requires or only supports
disabled interfaces, they will also get an exeption. The reality
is we need to suggest for them to look at enabling the interfaces
before looking at the default interface overrides, because logically
the brain jumps to setting a default before checking the interface
settings.

Change-Id: I50d4381e11da96cb7ae0ee8cbda18534380bd471
This commit is contained in:
Julia Kreger 2021-11-23 08:13:35 -08:00
parent f988c48aea
commit fdc6424de3
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
ironic/common/exception.py
View File

@ -245,7 +245,8 @@ class NoValidDefaultForInterface(InvalidParameterValue):
# because node_info could be an empty string. If node_info
# is not empty, it should start with a space.
_msg_fmt = _("For%(node_info)s hardware type '%(driver)s', no default "
"value found for %(interface_type)s interface.")
"value found for %(interface_type)s interface could be "
"determined. Please ensure the interfaces are enabled.")
class ImageNotFound(NotFound):