Skip adapter rename when set-name is not explicitly provided

When using network-config v2 with match.macaddress, cloudbase-init
always renames the adapter to the ethernets dict key name. This
causes failures on Windows because:

1. The WMI rename call (MSFT_NetAdapter.rename) is unreliable during
   early boot — the new name is not immediately queryable, causing
   "Network interface not found" errors (#101, #151)
2. The rename can hang for minutes on some Windows versions (#82)
3. Multiple adapters can cause name collisions

This aligns with cloud-init (Linux) behavior where the ethernets
dict key is just an opaque identifier when match is present, and
only set-name triggers an actual rename.

The fix checks link.id vs link.name: when they match (no set-name
was provided), the adapter is used with its current Windows name
instead of being renamed. When set-name is explicitly provided
(link.id != link.name), rename proceeds as before.

Signed-off-by: Max Makarov <maxpain@linux.com>

Author: maxpain

cloudbaseinit/plugins/common/networkconfig.py

@@ -198,11 +198,27 @@ def _process_physical_links(osutils, network_details):
                 link.mac_address)
 
             if adapter_name != link.name:
-                LOG.info(
-                    "Renaming network adapter \"%(old_name)s\" to "
-                    "\"%(new_name)s\"",
-                    {"old_name": adapter_name, "new_name": link.name})
-                osutils.rename_network_adapter(adapter_name, link.name)
+                if link.id != link.name:
+                    # set-name was explicitly provided, honor the rename
+                    LOG.info(
+                        "Renaming network adapter \"%(old_name)s\" to "
+                        "\"%(new_name)s\"",
+                        {"old_name": adapter_name, "new_name": link.name})
+                    osutils.rename_network_adapter(adapter_name, link.name)
+                else:
+                    # No set-name, use the actual adapter name to avoid
+                    # unreliable WMI rename (see issues #82, #101, #151)
+                    LOG.info(
+                        "Network adapter \"%(adapter_name)s\" found by "
+                        "MAC for config name \"%(config_name)s\", "
+                        "skipping rename",
+                        {"adapter_name": adapter_name,
+                         "config_name": link.name})
+                    for idx, net in enumerate(network_details.networks):
+                        if net.link == link.name:
+                            network_details.networks[idx] = net._replace(
+                                link=adapter_name)
+                    link = link._replace(name=adapter_name)
 
             NetworkConfigPlugin._process_link_common(osutils, link)
 

cloudbaseinit/tests/plugins/common/test_networkconfig.py

@@ -473,3 +473,96 @@ def test_execute_network_details_v2_ipv4_dns_list(self):
 
     def test_execute_network_details_v2_ipv6_dns_list(self):
         self._test_execute_network_details_v2(both_ipv6_dns_list=True)
+
+    @mock.patch("cloudbaseinit.osutils.factory.get_os_utils")
+    def test_execute_network_details_v2_no_rename_without_set_name(
+            self, mock_get_os_utils):
+        """When link.id == link.name (no set-name), skip adapter rename."""
+        link1 = network_model.Link(
+            id="eth0",
+            name="eth0",
+            type=network_model.LINK_TYPE_PHYSICAL,
+            enabled=True,
+            mac_address=u"00:00:00:00:00:01",
+            mtu=1500,
+            bond=None,
+            vlan_link=None,
+            vlan_id=None)
+
+        route1 = network_model.Route(
+            network_cidr=u"0.0.0.0/0",
+            gateway=u"10.0.0.254")
+
+        network1 = network_model.Network(
+            link="eth0",
+            address_cidr=u"10.0.0.1/24",
+            dns_nameservers=["10.0.0.1"],
+            routes=[route1])
+
+        network_details = network_model.NetworkDetailsV2(
+            links=[link1],
+            networks=[network1],
+            services=[])
+
+        service = mock.Mock()
+        service.get_network_details_v2.return_value = network_details
+
+        mock_os_utils = mock.Mock()
+        mock_get_os_utils.return_value = mock_os_utils
+        mock_os_utils.get_network_adapter_name_by_mac_address.return_value = \
+            "Ethernet 3"
+
+        plugin = networkconfig.NetworkConfigPlugin()
+        plugin.execute(service, {})
+
+        # Should NOT rename when link.id == link.name (no set-name)
+        mock_os_utils.rename_network_adapter.assert_not_called()
+
+        # Should use actual adapter name for all operations
+        mock_os_utils.enable_network_adapter.assert_called_once_with(
+            "Ethernet 3", True)
+        mock_os_utils.set_network_adapter_mtu.assert_called_once_with(
+            "Ethernet 3", 1500)
+        mock_os_utils.set_static_network_config.assert_called_once_with(
+            "Ethernet 3", "10.0.0.1", "24", "10.0.0.254", ["10.0.0.1"])
+
+    @mock.patch("cloudbaseinit.osutils.factory.get_os_utils")
+    def test_execute_network_details_v2_rename_with_set_name(
+            self, mock_get_os_utils):
+        """When link.id != link.name (set-name used), rename adapter."""
+        link1 = network_model.Link(
+            id="eth0",
+            name="my-nic",
+            type=network_model.LINK_TYPE_PHYSICAL,
+            enabled=True,
+            mac_address=u"00:00:00:00:00:01",
+            mtu=None,
+            bond=None,
+            vlan_link=None,
+            vlan_id=None)
+
+        network1 = network_model.Network(
+            link="my-nic",
+            address_cidr=u"10.0.0.1/24",
+            dns_nameservers=["10.0.0.1"],
+            routes=[])
+
+        network_details = network_model.NetworkDetailsV2(
+            links=[link1],
+            networks=[network1],
+            services=[])
+
+        service = mock.Mock()
+        service.get_network_details_v2.return_value = network_details
+
+        mock_os_utils = mock.Mock()
+        mock_get_os_utils.return_value = mock_os_utils
+        mock_os_utils.get_network_adapter_name_by_mac_address.return_value = \
+            "Ethernet"
+
+        plugin = networkconfig.NetworkConfigPlugin()
+        plugin.execute(service, {})
+
+        # Should rename when link.id != link.name (set-name was used)
+        mock_os_utils.rename_network_adapter.assert_called_once_with(
+            "Ethernet", "my-nic")