Fix network.py crash on unexpected NUMA node data (BugFix) (#2035)

Author: rodwsmith

providers/base/bin/network.py

@@ -213,6 +213,11 @@ def extract_core_list(self, line):
         cpu_list = line[colon + 1 :]
         core_list = []
         for core_range in cpu_list.split(","):
+            # Skip it if the CPU list for the NUMA node is empty....
+            core_range = core_range.strip()
+            if not core_range:
+                logging.error("Empty core range in line: %s", line)
+                continue
             # core_range should be of the form "a-b" or "a"
             range_list = core_range.split("-")
             if len(range_list) > 1:

providers/base/tests/test_network.py

@@ -25,6 +25,23 @@
 
 
 class IPerfPerfomanceTestTests(unittest.TestCase):
+    def test_extract_core_list_single(self):
+        """Tests parsing a NUMA node CPUs list with a single CPU."""
+        line = "NUMA node0 CPU(s): 0"
+        core_list = network.IPerfPerformanceTest.extract_core_list(None, line)
+        self.assertListEqual(core_list, [0])
+
+    def test_extract_core_list_range(self):
+        """Tests parsing a NUMA node CPUs list."""
+        line = "NUMA node0 CPU(s): 0-2"
+        core_list = network.IPerfPerformanceTest.extract_core_list(None, line)
+        self.assertListEqual(core_list, [0, 1, 2])
+
+    def test_extract_core_list_empty(self):
+        """Tests that parsing an empty NUMA node CPUs does not crash."""
+        line = "NUMA node0 CPU(s):"
+        core_list = network.IPerfPerformanceTest.extract_core_list(None, line)
+        self.assertListEqual(core_list, [])
 
     def test_find_numa_reports_node(self):
         with patch("builtins.open", mock_open(read_data="1")) as mo: