Repair promote gate tempdir cleanup failure after #1404 #1408

Closed
opened 2026-07-15 11:31:45 +00:00 by coilyco-ops · 6 comments
Member

Problem

After PR #1404 merged, promote.yml run 8505 / repository run #2235 failed in job gate-and-promote during make test.

Failure evidence:

run 8505 (#2235) failure promote.yml push main "Define broker and pre-flight check placement matrix (#1404)"
--- FAIL: TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns (0.00s)
    testing.go:1464: TempDir RemoveAll cleanup: unlinkat /tmp/TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns3709322320/001/.ward/agent-logs/dispatch/20260715T112759Z-18c27307bf7fbe23-unknown-coilyco-flight-deck-ward-795: directory not empty
FAIL github.com/coilyco-flight-deck/ward/cmd/ward 18.658s
make: *** [Makefile:178: test] Error 1

PR tests for #1404 and #1405 were green, but main promotion is red. This is separate from the release.yml failures tracked by #1402 and #1403.

Acceptance

  • Reproduce or inspect the tempdir cleanup failure in TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns.
  • Fix the test or production cleanup path so the dispatch-log temp directory is not left non-empty at test cleanup.
  • Add or preserve focused coverage for broker env restoration while launch runs.
  • Confirm go test ./cmd/ward and the promote gate pass, or leave exact blocker evidence.
  • Leave final evidence on this issue with the green run or remaining blocker.

Related: #1404, #1405, run 8505 / #2235.

## Problem After PR #1404 merged, promote.yml run 8505 / repository run #2235 failed in job gate-and-promote during make test. Failure evidence: ```text run 8505 (#2235) failure promote.yml push main "Define broker and pre-flight check placement matrix (#1404)" --- FAIL: TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns (0.00s) testing.go:1464: TempDir RemoveAll cleanup: unlinkat /tmp/TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns3709322320/001/.ward/agent-logs/dispatch/20260715T112759Z-18c27307bf7fbe23-unknown-coilyco-flight-deck-ward-795: directory not empty FAIL github.com/coilyco-flight-deck/ward/cmd/ward 18.658s make: *** [Makefile:178: test] Error 1 ``` PR tests for #1404 and #1405 were green, but main promotion is red. This is separate from the release.yml failures tracked by #1402 and #1403. ## Acceptance - Reproduce or inspect the tempdir cleanup failure in TestRunHostDispatchBrokerRequestClearsBrokerEnvWhileLaunchRuns. - Fix the test or production cleanup path so the dispatch-log temp directory is not left non-empty at test cleanup. - Add or preserve focused coverage for broker env restoration while launch runs. - Confirm go test ./cmd/ward and the promote gate pass, or leave exact blocker evidence. - Leave final evidence on this issue with the green run or remaining blocker. Related: #1404, #1405, run 8505 / #2235.
Author
Member

DISPATCH-DEFERRED: capacity blocked. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge at 2026-07-15T11:31Z. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T113149Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery as soon as a slot opens.

DISPATCH-DEFERRED: capacity blocked. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge at 2026-07-15T11:31Z. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T113149Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery as soon as a slot opens.
Author
Member

DISPATCH-DEFERRED: capacity blocked on retry. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge again at 2026-07-15T11:39Z after #1402 was relaunched. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T113952Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery at the next available slot.

DISPATCH-DEFERRED: capacity blocked on retry. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge again at 2026-07-15T11:39Z after #1402 was relaunched. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T113952Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery at the next available slot.
Author
Member

DISPATCH-DEFERRED: capacity blocked on retry. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge again at 2026-07-15T11:50Z after the lane appeared to have one free slot. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T115058Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery at the next available slot.

DISPATCH-DEFERRED: capacity blocked on retry. Attempted warded engineer coilyco-flight-deck/ward#1408 --harness codex --workflow pull-request-and-merge again at 2026-07-15T11:50Z after the lane appeared to have one free slot. Ward rejected the launch because coilyco-flight-deck/ward already had 3 active engineers at the repo limit. Broker evidence: repo engineer limit is reached for coilyco-flight-deck/ward: 3 active engineer(s) (limit 3). Dispatch log: /Users/kai/.ward/agent-logs/dispatch/20260715T115058Z-director-codex-jp85-coilyco-flight-deck-ward-1408.log. Retry this P1 promote recovery at the next available slot.
Author
Member

WARDED_WORKFLOW: reservation-held

reservation details

Holder: launch intent for container engineer-codex-ward-1408 on host kais-macbook-pro-2.local.

Accepted by ward agent --harness codex (reserved 2026-07-15T12:00:01Z). Concurrent ward agent runs are blocked until this intent becomes visible or the intent is released. The stale-intent fallback is still TTL-bounded (3h TTL). --override-reservation overrides.

Do not comment on or edit this issue to steer the run while it is reserved. The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a new issue, dispatched fresh. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494).

run seed context — what this run is carrying (ward#609)
  • Resolved: coilyco-flight-deck/ward#1408 · branch issue-1408 · harness codex · workflow pull-request-and-merge
  • Run: engineer-codex-ward-1408 · ward v0.710.0 · dispatched 2026-07-15T11:59:56Z
  • Comment thread: 3 included in the pre-flight read, 0 stripped (ward's own automated comments).

Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.710.0).

— Codex, via ward agent

<!-- ward-agent-reservation --> WARDED_WORKFLOW: reservation-held <details><summary>reservation details</summary> Holder: launch intent for container `engineer-codex-ward-1408` on host `kais-macbook-pro-2.local`. Accepted by `ward agent --harness codex` (reserved 2026-07-15T12:00:01Z). Concurrent `ward agent` runs are blocked until this intent becomes visible or the intent is released. The stale-intent fallback is still TTL-bounded (3h TTL). `--override-reservation` overrides. **Do not comment on or edit this issue to steer the run while it is reserved.** The engineer seeded the body once at launch and never re-reads it, so a comment or edit reaches only human readers, never the running engineer. A correction goes to a **new issue, dispatched fresh**. That is the only channel that reaches a run in flight. Where the forge supports it, ward locks this conversation to make that a road-block rather than a convention (ward#494). <details><summary>run seed context — what this run is carrying (ward#609)</summary> - **Resolved:** `coilyco-flight-deck/ward#1408` · branch `issue-1408` · harness `codex` · workflow `pull-request-and-merge` - **Run:** `engineer-codex-ward-1408` · ward `v0.710.0` · dispatched `2026-07-15T11:59:56Z` - **Comment thread:** 3 included in the pre-flight read, 0 stripped (ward's own automated comments). - included: @coilyco-ops (2026-07-15T11:32:04Z), @coilyco-ops (2026-07-15T11:40:10Z), @coilyco-ops (2026-07-15T11:51:16Z) Static container doctrine and seed boilerplate are identical every run and omitted here (they ride ward v0.710.0). </details> </details> <!-- ward-agent-signature --> — Codex, via `ward agent`
Author
Member

WARDED_WORKFLOW: #1421

details

workflow: pull-request-and-merge; review summary: review gate intentionally skipped because temporary ward default pending brokered QA
felt: narrow, mechanical, and stable after the regression assertion
confidence: high
surprise: the live tree was already green, so the fix landed as a regression lock rather than a codepath repair
follow-up: none

WARDED_WORKFLOW: https://forgejo.coilysiren.me/coilyco-flight-deck/ward/pulls/1421 <details><summary>details</summary> workflow: pull-request-and-merge; review summary: review gate intentionally skipped because temporary ward default pending brokered QA felt: narrow, mechanical, and stable after the regression assertion confidence: high surprise: the live tree was already green, so the fix landed as a regression lock rather than a codepath repair follow-up: none </details>
Author
Member

QA FAIL after PR #1421 / merge commit c34d7161cc.

PR #1421 merged with green PR test run 8529, but post-merge promote run 8530 / repo run #2252 failed in gate-and-promote during make test:

run 8530 (#2252) failure promote.yml push main "Repair promote gate tempdir cleanup failure after #1404 (#1421)"

That means #1408's acceptance is not met yet: promote.yml has not been shown green after the repair. Filed follow-up #1423 with identical headless/P1 milestone queueing for the post-#1421 promote failure. #1422 is already running for the earlier post-#1407 promote failure, but it was dispatched before run 8530 existed.

QA FAIL after PR #1421 / merge commit c34d7161cc72d872abe4cbf626177060c6da9b92. PR #1421 merged with green PR test run 8529, but post-merge promote run 8530 / repo run #2252 failed in gate-and-promote during make test: `run 8530 (#2252) failure promote.yml push main "Repair promote gate tempdir cleanup failure after #1404 (#1421)"` That means #1408's acceptance is not met yet: promote.yml has not been shown green after the repair. Filed follow-up #1423 with identical headless/P1 milestone queueing for the post-#1421 promote failure. #1422 is already running for the earlier post-#1407 promote failure, but it was dispatched before run 8530 existed.
Sign in to join this conversation.
No project
No assignees
1 participant
Notifications
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#1408
No description provided.