Repository navigation
actions: runner daemon (T2.1) and tier2 package - #56
Open
calvin-archastro wants to merge 1 commit into
Open
calvin-archastro wants to merge 1 commit into
calvin-archastro wants to merge 1 commit into
Conversation
internal/tier2: Tier 2 record types, key paths, Store interface, PailStore (shells out to pail obj), MemStore, and a fake pail obj for tests. internal/runner: queue scan, lease claim/takeover/refresh, cancel, progress, log chunks, results; act executor with checkout swapped for pail clone --at. docs/actions.md 5.1: optional job keys on the queue entry. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Merge order: after the
pail objPR (feat/obj-cli) for real use. The runner shells out topail obj; the integration test falls back to a contract fake (tier2/fakepail) when the pail binary has noobj, so CI is green either way.What
Task T2.1 (runner daemon) and minimal T2.3:
pail-actions runner --store URL --labels a,b [--name ID] [--once], speaking the bucket protocol of docs/actions.md sections 5 and 8.actions/internal/tier2: Go types for every Tier 2 record, path helpers (QueuePath,LeasePath,ResultPath, ...),Storeinterface,PailStore(pail obj),MemStore(in-memory, with fault injection),FakeClock,fakepail(shell stand-in forpail objin tests).actions/internal/runner: poll, claim (create or takeover seq+1 by version match, never delete+recreate), 15 s refresh, loss detection (stops all writes, cancels act), cancel marker, progress, log chunks (64 KB or 2 s) then full.log, result atresults/.../<seq>,runners/<id>.jsonheartbeat. act runs a one-job workflow synthesized from the entry;actions/checkoutbecomespail clone --at; act github context comes from the entry.actions/RUNNER.md: usage, tests, manual docker command and result, what is left.Contract change
docs/actions.md section 5.1 only adds optional queue entry keys (
runs_on,name,container,services,defaults;github.run_number|actor|head_ref|base_ref) and says steps are the YAML step objects verbatim. No existing field changes.Tests
--oncewaits and heartbeats.pail obj, same suite as MemStore.Not done
Secrets (ignored with a log line), job outputs, artifacts/cache, scaler (T2.2). See RUNNER.md.
🤖 Generated with Claude Code