fix(drivers): address review comments on delete_machine idempotency

- Log at debug level when a domain is already gone during delete, so
  the missing-domain path stays traceable instead of silent.
- Add a regression test confirming volume cleanup still runs via
  list_volumes when the domain was already gone (previously only
  the no-crash case was covered).

Author: VasilyStepanov

exordos_core/compute/pool/drivers/libvirt.py

@@ -1340,6 +1340,10 @@ def delete_machine(
         except libvirt.libvirtError as e:
             if e.get_error_code() != libvirt.VIR_ERR_NO_DOMAIN:
                 raise
+            LOG.debug(
+                "Domain for machine %s not found, assuming already deleted",
+                machine.uuid,
+            )
             domain = None
 
         if domain is not None:

exordos_core/tests/unit/compute/pool/drivers/test_libvirt.py

@@ -15,6 +15,7 @@
 #    under the License.
 
 import uuid as sys_uuid
+from unittest import mock
 from xml.dom import minidom
 from xml.etree import ElementTree as ET
 
@@ -117,3 +118,22 @@ def test_is_idempotent_when_the_domain_is_already_gone(self):
 
         # Must not raise, even though no such domain was ever defined.
         driver.delete_machine(machine, delete_volumes=False)
+
+    def test_volume_cleanup_still_runs_when_the_domain_is_already_gone(self):
+        driver = _local_driver()
+        machine = models.Machine(
+            uuid=sys_uuid.uuid4(),
+            project_id=sys_uuid.uuid4(),
+            name="never-existed",
+            cores=1,
+            ram=512,
+        )
+
+        # The missing-domain path must fall through to volume cleanup,
+        # not skip it.
+        with mock.patch.object(
+            driver, "list_volumes", return_value=[]
+        ) as mock_list_volumes:
+            driver.delete_machine(machine, delete_volumes=True)
+
+        mock_list_volumes.assert_called_once_with(machine)