Fix broker cleanup on Docker name conflicts #1306

Merged
coilyco-ops merged 1 commit from issue-1282 into main 2026-07-15 03:44:51 +00:00
Owner

This change keeps the broker from treating a Docker duplicate-name refusal as a reason to stop the live engineer container. When a launch collides with an existing container name, the broker now records the failure, unlocks the issue, and leaves the existing run alone instead of trying to clean it up by name.

  • Added a duplicate-name detector in the broker failure path.
  • Added a regression test that proves the stop call is skipped for the conflict case.
  • Validation: ward exec test and pre-commit run --all-files.

closes #1282
ward.workflow: pull-request-and-merge

This change keeps the broker from treating a Docker duplicate-name refusal as a reason to stop the live engineer container. When a launch collides with an existing container name, the broker now records the failure, unlocks the issue, and leaves the existing run alone instead of trying to clean it up by name. - Added a duplicate-name detector in the broker failure path. - Added a regression test that proves the stop call is skipped for the conflict case. - Validation: `ward exec test` and `pre-commit run --all-files`. closes #1282 ward.workflow: pull-request-and-merge
Fix broker launch conflict cleanup
All checks were successful
test / test (pull_request) Successful in 48s
988d2b0cab
closes #1282
Commenting is not possible because the repository is archived.
No reviewers
No milestone
No project
No assignees
1 participant
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
coilyco-flight-deck/ward!1306
No description provided.