tidy: project finished backup jobs onto host row + smoke doc tweaks #20

Merged

steve merged 1 commits from tidy-up-last-backup-projection into main 2026-05-07 17:58:16 +01:00

Conversation 0 Commits 1 Files Changed 3

+32 -13

steve commented 2026-05-07 17:55:39 +01:00

Owner

Copy Link

Copy Source

Summary

• Fix: the WS `job.finished` handler had a switch on `job.kind` covering `init` (writes `hosts.repo_status`) but silently dropped `backup` terminations. The dashboard's "Last backup" column reads `hosts.last_backup_at` / `last_backup_status` and was therefore stale until the next snapshot-list push refreshed something else. Add a `SetHostLastBackup` store method and a `case api.JobBackup:` arm.
• Doc: CLAUDE.md restage block referenced `/tmp/rm-smoke` (the original smoke env path); the dev box runs out of `$HOME/smoke`. Updated to match.

Test plan

• Trigger a backup on any host. After completion, check that the dashboard "Last backup" column updates without waiting for a snapshot-list refresh.
• `go test ./internal/store/ ./internal/server/ws/` green (covered by existing tests; the new `SetHostLastBackup` is a one-line UPDATE).

## Summary - Fix: the WS \`job.finished\` handler had a switch on \`job.kind\` covering \`init\` (writes \`hosts.repo_status\`) but silently dropped \`backup\` terminations. The dashboard's "Last backup" column reads \`hosts.last_backup_at\` / \`last_backup_status\` and was therefore stale until the next snapshot-list push refreshed something else. Add a \`SetHostLastBackup\` store method and a \`case api.JobBackup:\` arm. - Doc: CLAUDE.md restage block referenced \`/tmp/rm-smoke\` (the original smoke env path); the dev box runs out of \`\$HOME/smoke\`. Updated to match. ## Test plan - [ ] Trigger a backup on any host. After completion, check that the dashboard "Last backup" column updates without waiting for a snapshot-list refresh. - [ ] \`go test ./internal/store/ ./internal/server/ws/\` green (covered by existing tests; the new \`SetHostLastBackup\` is a one-line UPDATE).

steve added 1 commit 2026-05-07 17:55:40 +01:00

fix: project finished backup jobs onto host row + smoke path tweaks ... 711d5e964c

The dashboard's 'Last backup' column reads hosts.last_backup_at /
last_backup_status, but the WS handler only updated hosts.repo_status
on job.finished — backup terminations were silently dropped. Add a
SetHostLastBackup store method and call it from the same job.finished
switch that already handles init jobs.

Also: CLAUDE.md restage block uses /tmp/rm-smoke (the original
default) but the actual dev env runs out of $HOME/smoke. Update the
paths in the doc to match.

steve merged commit 9c098e773b into main 2026-05-07 17:58:16 +01:00

steve referenced this issue from a commit 2026-05-09 14:23:46 +01:00

Merge pull request 'tidy: project finished backup jobs onto host row + smoke doc tweaks' (#20) from tidy-up-last-backup-projection into main

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

steve

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: steve/restic-manager#20