Repository navigation
fleet: unified schedule registry — Muse cron + OpenCode + Jules (#4988) - #5166
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 52 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. CodeRabbit Verify
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f02decb72d
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| | `laptop-autocommit-hatch-personal.timer` | `OnCalendar=*-*-*` | `laptop-autocommit-hatch-personal.service` | laptop (user systemd) | not recorded | not recorded | not recorded | 2026-11-03 | | ||
| | `laptop-autocommit-hatch.timer` | `OnCalendar=*-*-*` | `laptop-autocommit-hatch.service` | laptop (user systemd) | not recorded | not recorded | not recorded | 2026-11-03 | | ||
| | `laptop-autocommit-openclaw.timer` | `OnCalendar=*-*-*` | `laptop-autocommit-openclaw.service` | laptop (user systemd) | not recorded | not recorded | not recorded | 2026-11-03 | |
There was a problem hiding this comment.
Preserve the time component of systemd calendar schedules
The calendar-based rows omit the time component, changing the documented cadence rather than merely abbreviating it. For example, laptop-autocommit is configured with *-*-* *:*:00 in nixos/home/laptop-autocommit.nix:51 (every minute), but these rows show *-*-*, which systemd-analyze calendar normalizes to *-*-* 00:00:00 (daily at midnight). The same loss affects the hourly worktree reaper and 15-minute quota gatherer elsewhere in this table. Since this registry is explicitly an input to the placement audit, these entries misrepresent execution frequency and workload. Preserve the complete calendar expressions when collecting the timer specs.
Useful? React with 👍 / 👎.
|
|
||
| ### 2a. OpenChamber scheduledTasks (16 tasks, laptop stores) | ||
|
|
||
| Route/pool for queue-lane tasks is the repo-health queue named in the task name; the claimant/executor task drives the #3026 execution-lane system. Prompt/brief pointers are the task records in the named project store. |
There was a problem hiding this comment.
Record execution routes separately from work queues
The repo-health queue identifies the work scope, not the execution route or quota pool, so it cannot supply the route/pool field required by the Fields section. This information is available for at least the claimant: pm-stack/opencode-claimant/install.mjs:21-26,75-84 sets its execution route to opencode-route / workhorse, but neither its registry row nor this explanation records that route. Omitting execution routes prevents the intended placement/shared-pool audit from identifying which routes recurring model jobs consume. Add the actual execution route/pool to the records, retain queue names as scope, and explicitly mark unavailable routes as not recorded.
Useful? React with 👍 / 👎.
For #4988 (task 8792) — the unified fleet schedule registry from the #2024 placement framework (B2).
What
doc/fleet-schedule-registry.md— one record for every currently-known schedule on all three surfaces, populated from live sources verified 2026-11-03:OnCalendar/OnUnitActiveUSecspecs — the OS-timer surface implement: fleet OpenCode scheduled-task pattern — systemd timers + opencode run with per-job route #2232 builds on (still open, listed as-is).not recorded, never guessed.AGENTS.md— the same-pass rule written into the repo's runbook text: creating/changing/retiring a schedule on any surface updates the registry in the same pass; the rule is also stated in the registry's own header.No schedule's cadence or rate was changed (per the packet's scope exclusion).
Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.