Repository navigation
fix(lab): require daemon lease identity for admission - #15518
Conversation
Homeboy Results —
|
Current-head CI updateFor head |
CI UpdateFor current head |
Current-head CI result\nFor head
|
|
Retained during open-PR landing review: the lease fix is still relevant. Refreshed the branch with current main, including the green-baseline repair in #15571. This remains draft: real remote SSH-tunnel/shared-authority coverage and terminal remote-job replay are explicitly incomplete in the existing evidence. A green rerun alone will not establish those acceptance criteria. AI assistance: OpenAI gpt-6.1-sol via OpenCode reviewed source, existing discussion and CI logs, and refreshed the branch under Chris Huber’s instruction to land ready PRs or close superseded work. |
|
Final landing disposition: retain as relevant to #15505, but keep draft. Workflow 37487649076 reports explicit_lab_route_persists_the_verified_lab_outcome_through_detached_cook_lifecycle as candidate-only after isolated retries; its other candidate-only daemon assertion passed on retry and was classified flaky. The PR also explicitly lacks real remote SSH-tunnel/shared-authority and terminal remote-job replay acceptance. Those proofs and a green exact-head verdict remain necessary. This work is unfinished, rather than obsolete. AI assistance: OpenAI gpt-6.1-sol via OpenCode reviewed the tracker, implementation, retained evidence and authoritative attribution logs under Chris Huber’s landing instruction. |
…ontrol-plane # Conflicts: # crates/homeboy-lab-runner/src/runners.rs
chubes4
left a comment
There was a problem hiding this comment.
Final parent review accepts exact head 0dc210f. The diff publishes controller setup progress with the initial queued record, recognizes the actual pre-provider phases in the canonical stale-run fence, prevents replay claim inheritance/consumption by startup reconcilers, and uses one lease/freshness verdict for admission/status. Retained public CLI regression with real disposable daemon/store/broker proves no job routes before the withheld exact lease is republished, then one detached replay reaches terminal Lab custody with a persisted outcome. Failure evidence is retained and temporary diagnostics were removed. All authoritative current-head Candidate, Baseline, aggregate Test, Lint, Rustfmt, Plan and binary-preparation checks passed; relevant full/focused local suites and baseline comparisons are recorded. Shipping the reviewed repair under Chris Huber’s keep-shipping direction. The earlier PR body’s pending-runtime limitations are superseded by the final worker proof and this exact-head review; no operator service deployment is performed here. AI assistance: OpenAI gpt-6.1-sol via OpenCode independently reviewed production changes, real runtime/store test coverage and completed exact-head checks; OpenAI gpt-6-luna via OpenCode implemented and verified the repair.
Summary
RunnerDaemonLeaseVerdictin the status/doctor admission summary with generation identity/source, session observation timestamp and age, freshness/lease state, and the same next action admission uses.Closes #15505
Verification
cargo test --target x86_64-unknown-linux-gnu --test runner_service_observation -- --test-threads=1: 2 passed (disk-backed test homes).cargo test --test daemon_ensure_running_cli -- --test-threads=1: 5 passed.cargo fmt --all -- --checkandgit diff --checkpassed.37402652756for19c3608465: Rustfmt and Plan passed; Lint and candidate-binary preparation are pending. The previousfd6324a925Candidate/Baseline phases passed but the final Test wrapper could not reconcile missing baseline phase artifacts.Acceptance remains incomplete: service integration uses localhost loopback rather than a remote SSH tunnel, so shared remote tunnel authority/rotation is unproven; terminal explicit-run-ID protection is tested separately from terminal remote service replay. No operator runner/service was contacted. Detailed evidence:
evidence/homeboy-15505/continued-evidence.md. Keep draft pending these criteria and current-head CI.AI Disclosure
OpenAI gpt-6-luna via OpenCode analyzed runner/session/generation-store code, implemented fail-closed lease handling, a canonical generation/age verdict, lock diagnostics, and real-daemon/service-managed regression coverage. A human-authorized operator supplied the tracker and authorized direct-runtime execution. AI did not contact an operator runner or claim the remaining SSH-tunnel criteria.