fix(garm): exact identity and dispatch occupancy for v0.2.1-nddev.94 - #431
Conversation
Bind stale-job mutation to check-run identity, take scaling from the live MESSAGE session, and stop request-less JobAssigned tokens from blocking a different GUID that GitHub actually made available. Replay and replaced sessions cannot refresh idle freshness. This is a source/artifact candidate, not a fleet rollout. Signed-off-by: rldyourmnd <danil@nddev.it.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Reject omitted or changing GitHub list totals, bind check_run_url by path ID, cancel the listener context when Start fails, drop idle TryLock before the next candidate, and stop the build container before deleting its bind source. Pin the frozen two-cache artifact. Source/artifact candidate only. Co-authored-by: Cursor <cursoragent@cursor.com> Signed-off-by: rldyourmnd <danil@nddev.it.com> Co-authored-by: Cursor <cursoragent@cursor.com>
Quota-blocked JobAvailable was demoting every request-less reservation, then excluding that waiter from readmission, so a foreign repo could idle a safe slot. Yield only occupancy a dispatchable candidate can actually use and keep A/GUID/FIFO. Signed-off-by: rldyourmnd <danil@nddev.it.com> Co-authored-by: Cursor <cursoragent@cursor.com>
R02 admission fixHEAD is now Isolated regression against previous HEAD Fix: yield only occupancy a currently admissible dispatchable job can use. GUID/FIFO kept. Focused and race This is still a source candidate. Do not merge or roll out from this comment. Previous artifact |
Bind check_run_url to https origin and repository path, keep 404 fail-closed without access-refusal backoff, yield request-less occupancy only to dispatchable jobs, and pin the two-cache artifact 75ec2a259092859bbb42b448f99b549bc33606649b4c953bf4292b006c7652a9. Preserve the build tree when container stop is not proven. Signed-off-by: rldyourmnd <danil@nddev.it.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
Frozen .94 candidate after R02 admissible-yield and R03 identity/build review. Source: 913b483 |
|
Self-review of HEAD 913b483 / artifact 75ec2a259092859bbb42b448f99b549bc33606649b4c953bf4292b006c7652a9. yieldRequestlessOccupancy demotes a request-less assigned intent only when a dispatchable queued job would actually fit after that demote. TestQueueCoordinatorQuotaBlockedAvailableMustNotEvictUsefulBootstrap covers the audit regression. check_run_url is fail-closed to https repos/{owner}/{repo}/check-runs/{id} bound to the check URL. 404 does not start the 15m 403/429 backoff. Demand/messageID 0 after lock does not authorize RemoveRunner. Official build preserves the bind-mounted source tree when container stop is unproven. .92/.93 pins are untouched. Merge is source integration only. Not a fleet rollout. Live identity remains .92 until R04 exact-pin + /proc/PID/exe readback. Approve. |
Summary
Test plan