diff options
| author | Julia Kreger <juliaashleykreger@gmail.com> | 2021-11-23 08:13:35 -0800 |
|---|---|---|
| committer | Riccardo Pittau <elfosardo@gmail.com> | 2021-11-30 08:12:04 +0000 |
| commit | f561d1ccb451bb3a3f3ee9134f312f2c023628ed (patch) | |
| tree | fc2ec5de710d74d78dd648f65084f8f75e4a8c1d | |
| parent | fa4cc785760e2d9d3fdebf1db8662aa30105f884 (diff) | |
| download | ironic-f561d1ccb451bb3a3f3ee9134f312f2c023628ed.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
(cherry picked from commit fdc6424de3ad4beeeb95d348f422af00e03d1892)
| -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 75c7b02c5..a4eed77ff 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): |