From b20a70646f1b1dd985ab9fef8a9d4e7d89a8da9c Mon Sep 17 00:00:00 2001 From: Kevin Veen-Birkenbach Date: Mon, 27 Jul 2026 12:25:13 +0200 Subject: [PATCH] fix(swarm-test): give the sandbox tier a second node in the lab Dedicating a single worker to the sandbox left the drain step with nowhere to put the agent: draining the only labelled node failed with `no suitable node (scheduling ...)`. Two labelled workers are also what the requirement asks for, so the earlier single-worker choice traded one defect for another. The cost is deliberate: the inverse placement constraint pushes every non-sandbox service off both workers, so the stack host carries the rest of the platform. That is the topology the requirement describes, and whether the stack host holds up under it is what the next swarm run measures. Co-Authored-By: Claude Opus 5 --- utils/tests/swarm/extend_inventory.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/utils/tests/swarm/extend_inventory.py b/utils/tests/swarm/extend_inventory.py index 38d6d328b5..46999d27d7 100755 --- a/utils/tests/swarm/extend_inventory.py +++ b/utils/tests/swarm/extend_inventory.py @@ -86,7 +86,7 @@ def main() -> int: closure = derive_includes(app_id) group_hosts = _host_topology(app_id) + _placement_dep_groups(app_id) if "svc-virt-kata" in closure: - group_hosts.extend(("svc-virt-kata", host) for host in (_MANAGER, _WORKERS[-1])) + group_hosts.extend(("svc-virt-kata", host) for host in (_MANAGER, *_WORKERS)) group_hosts.extend( repo_placements( app_closure=closure,