summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--os_net_config/objects.py2
-rw-r--r--os_net_config/tests/test_objects.py6
2 files changed, 8 insertions, 0 deletions
diff --git a/os_net_config/objects.py b/os_net_config/objects.py
index 4f3e0d5..c0c1719 100644
--- a/os_net_config/objects.py
+++ b/os_net_config/objects.py
@@ -98,6 +98,8 @@ def _numbered_nics(nic_mapping=None):
_NUMBERED_NICS[nic_alias] = nic_mapped
logger.info("%s mapped to: %s" % (nic_alias, nic_mapped))
+ if not _NUMBERED_NICS:
+ logger.warning('No active nics found.')
return _NUMBERED_NICS
diff --git a/os_net_config/tests/test_objects.py b/os_net_config/tests/test_objects.py
index d3e41df..a4a5f8a 100644
--- a/os_net_config/tests/test_objects.py
+++ b/os_net_config/tests/test_objects.py
@@ -569,3 +569,9 @@ class TestNumberedNicsMapping(base.TestCase):
mapping = {'nic1': '12:34:56:de:f0:12', 'nic2': '12:34:56:78:9a:bc'}
expected = {'nic1': 'em2', 'nic2': 'em1'}
self.assertEqual(expected, objects._numbered_nics(nic_mapping=mapping))
+
+ def test_numbered_nics_no_active(self):
+ self._stub_active_nics([])
+ expected = {}
+ # This only emits a warning, so it should still work
+ self.assertEqual(expected, objects._numbered_nics())