diff options
| author | Julia Kreger <juliaashleykreger@gmail.com> | 2021-11-23 08:13:35 -0800 |
|---|---|---|
| committer | Julia Kreger <juliaashleykreger@gmail.com> | 2021-11-23 20:40:51 +0000 |
| commit | fdc6424de3ad4beeeb95d348f422af00e03d1892 (patch) | |
| tree | ec6fbe329338ef8251e33f1269007790d266eec8 | |
| parent | f988c48aea018c8268763433883648879788027c (diff) | |
| download | ironic-fdc6424de3ad4beeeb95d348f422af00e03d1892.tar.gz | |
Clarify driver load error message
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
| -rw-r--r-- | ironic/common/exception.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ironic/common/exception.py b/ironic/common/exception.py index 1f19d6e6b..9d9420f9d 100644 --- a/ironic/common/exception.py +++ b/ironic/common/exception.py @@ -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): |