Repository navigation
auto-discover consumes the shared inspect API #34
Description
35 — auto-discover consumes the shared inspect API
What to build: auto-discover drops its local DockerClient and bare ContainerInfo, uses the lab-lib types, and deletes the CLI-shelled get_container_ip in favor of the bollard network-settings read in the shared inspect. container_matches and the sync tests cover the grown ContainerInfo; behavior is unchanged.
Blocked by: #33 — the shared inspection module must land first.
Status: ready-for-agent
- Local
DockerClient+ bareContainerInforemoved; lab-lib types used throughout -
get_container_ipCLI path deleted; the container IP comes from the shared inspect (bollard network settings) -
container_matchesand sync tests cover the grownContainerInfo; behavior unchanged - auto-discover lib suite green
Activity
refactor(ll): add shared docker container inspection module
refactor(ad): consume shared lab-lib inspect API for container IPs
FAZuH commentedon Oct 4, 2026
This work is already landed on
v0.2(pointer at merge3dc79fa).feat/c4-port-authorityandfeat/c5-inspectionwere rebased ontov0.2; every commit was a patch-equivalent already present there, so both branches linearized down tov0.2cleanly, the tree was verified (PortAssignmentsabsent, shared lab-lib inspect module incrates/lab-lib/src/docker.rs, both daemons consuming it), and./dev.sh testpassed 431/0. Closed by hand because the merge history does not fire a keyword from the non-default branch.