Fix find_resource_slot live mode: extract host from label_allocations

The host (instance_parent) for reservations was only checked in
sliver.get_labels(), which stores request labels and typically does
not contain the allocated host. The actual host assignment lives in
sliver.get_label_allocations(). This caused all reservations to have
host=None, silently skipping resource subtraction and making reserved
resources appear available.

Now checks label_allocations first, falling back to labels. Also adds
warning logging when host is None and debug logging for each
reservations extracted data.

Author: kthare10

fabric_cf/orchestrator/core/orchestrator_handler.py

@@ -1388,8 +1388,18 @@ def _find_slot_live(self, *, body: dict, token: str) -> dict:
                     r_start = ActorClock.from_milliseconds(milli_seconds=r.get_start())
                     r_end = ActorClock.from_milliseconds(milli_seconds=r.get_end())
                     host = None
-                    if sliver.get_labels() and sliver.get_labels().instance_parent:
-                        host = sliver.get_labels().instance_parent
+                    label_allocs = sliver.get_label_allocations()
+                    labels = sliver.get_labels()
+                    if label_allocs and getattr(label_allocs, 'instance_parent', None):
+                        host = label_allocs.instance_parent
+                    elif labels and getattr(labels, 'instance_parent', None):
+                        host = labels.instance_parent
+                    if host is None:
+                        self.logger.warning(
+                            f"find_resource_slot: reservation {r.get_reservation_id()} "
+                            f"has no instance_parent in labels or label_allocations; "
+                            f"its resources will not be subtracted from host capacity"
+                        )
                     alloc = sliver.capacity_allocations
                     caps = sliver.capacities
                     cores = (getattr(alloc, 'core', 0) or 0) if alloc else 0
@@ -1398,14 +1408,20 @@ def _find_slot_live(self, *, body: dict, token: str) -> dict:
                     ram = ram or ((getattr(caps, 'ram', 0) or 0) if caps else 0)
                     disk = (getattr(alloc, 'disk', 0) or 0) if alloc else 0
                     disk = disk or ((getattr(caps, 'disk', 0) or 0) if caps else 0)
+                    comps = self._extract_reservation_components(sliver)
+                    self.logger.debug(
+                        f"find_resource_slot: reservation {r.get_reservation_id()} "
+                        f"host={host} cores={cores} ram={ram} disk={disk} "
+                        f"components={comps} lease=[{r_start}, {r_end}]"
+                    )
                     compute_reservations.append({
                         "host": host,
                         "lease_start": r_start,
                         "lease_end": r_end,
                         "cores": cores,
                         "ram": ram,
                         "disk": disk,
-                        "components": self._extract_reservation_components(sliver),
+                        "components": comps,
                     })
 
         # Query network service reservations (for links and/or facility ports)