net: when selecting a network device, use natural sort order

The code deciding which interface to choose as the default to request the
IP address through DHCP does not sort the interfaces correctly. On Ubuntu
Xenial images for example, the interfaces are named ens1, ens2, ens3...,
ens11, ... depending on the pci bus address. The python sorting will list
'ens11' before 'ens3' for example despite the fact that 'ens3' should be
before 'ens11'.

This patch address this issue and sort the interface names according to a
human sorting.

Signed-off-by: Marc-Aurèle Brothier <m@brothier.org>

1 files changed, 14 insertions, 0 deletions

diff --git a/tests/unittests/test_net.py b/tests/unittests/test_net.py
index 5f7e902a..0000b075 100644
--- a/tests/unittests/test_net.py
+++ b/tests/unittests/test_net.py
@@ -1,6 +1,7 @@
 # This file is part of cloud-init. See LICENSE file for license information.
 
 from cloudinit import net
+from cloudinit.net import _natural_sort_key
 from cloudinit.net import cmdline
 from cloudinit.net import eni
 from cloudinit.net import netplan
@@ -1659,6 +1660,19 @@ class TestGetInterfacesByMac(CiTestCase):
         self.assertEqual('lo', ret[empty_mac])
 
 
+class TestInterfacesSorting(CiTestCase):
+
+    def test_natural_order(self):
+        data = ['ens5', 'ens6', 'ens3', 'ens20', 'ens13', 'ens2']
+        self.assertEqual(
+            sorted(data, key=_natural_sort_key),
+            ['ens2', 'ens3', 'ens5', 'ens6', 'ens13', 'ens20'])
+        data2 = ['enp2s0', 'enp2s3', 'enp0s3', 'enp0s13', 'enp0s8', 'enp1s2']
+        self.assertEqual(
+            sorted(data2, key=_natural_sort_key),
+            ['enp0s3', 'enp0s8', 'enp0s13', 'enp1s2', 'enp2s0', 'enp2s3'])
+
+
 def _gzip_data(data):
     with io.BytesIO() as iobuf:
         gzfp = gzip.GzipFile(mode="wb", fileobj=iobuf)