JORGE SASIAIN · 91fe5d65 · 91fe5d65
--- a/gso/services/netbox_client.py
+++ b/gso/services/netbox_client.py
+
+        This will delete a device and all interfaces that belong to it
+        """
+        device = self.get_device_by_name(device_name)
+        device.delete()
+        return
+
+    def delete_interface(self, device_name: str, iface_name: str) -> None:
+        """Delete interface by name and by device name.
+
+        This will delete an interface even if another one has it as LAG interface
+        (and will erase the LAG interface field of the latter)
+        """
+        interface = self.get_interface_by_name_and_by_device_id(
+            iface_name=iface_name, device_id=self.get_device_by_name(device_name).id
+        )
--- a/gso/services/netbox_client.py
+++ b/gso/services/netbox_client.py
+        # Get the lag attached to the physical interface and ensure it is the expected one
+        lag = self.get_interface_by_name_and_by_device_id(iface_name=lag_name, device_id=device.id)
+        if not iface.lag:
+            raise WorkflowStateError(
+                f"The interface: {iface_name} on device: {device_name} does not belong to any LAG."
+            )
+        if iface.lag.id != lag.id:
+            raise WorkflowStateError(
+                f"The interface: {iface_name} on device: {device_name} belongs to a different LAG: {lag.name}."
+            )
+
+        # Remove physical interface from lag (that is, remove the lag reference in the physical interface)
+        iface.lag = None
+
+        # Update physical interface
+        iface.save()
--- a/gso/services/netbox_client.py
+++ b/gso/services/netbox_client.py

        return interface

-    def allocate_interface(self, device_name: str, iface_name: str) -> Interfaces:
-        """Allocate an interface by marking it as connected."""
-
-        device = self.get_device_by_name(device_name)
-        interface = self.netbox.dcim.interfaces.get(device_id=device.id, name=iface_name)
+    def unreserve_interface(self, device_name: str, iface_name: str) -> Interfaces: