Commit Graph
216 Commits
Author SHA1 Message Date
osobh bf0f3be742 Merge pull request 'Polish: extract src/lib.rs' (#87) from polish-lib-split into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 3s
2026-07-14 20:14:19 +00:00
Omar Sobh 0e5e5982d8 Polish: extract src/lib.rs so bins reuse a proper library crate
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 2s
Historically every bin (claw-store, claw-cargo, claw-fuse)
re-declared the same 13-line 'mod ...' block at its own root.
Working but fragile — a new bin (or a new module) needed edits
in N+1 places and stayed one edit-slip away from silently
dropping something.

Now: src/lib.rs owns the shared module tree. Cargo.toml gets a
[lib] entry so cargo picks it up. claw_fuse.rs migrated as the
proof-of-concept — 13 mod lines → 4 use lines.

claw-store + claw-cargo bins still use their internal 'mod ...'
blocks and 'crate::...' paths — migrating them is mechanical
but noisy; deferred to a follow-on so this PR stays reviewable.
Both bins + tests continue to build unchanged.

387 tests pass.
2026-07-14 13:14:14 -07:00
osobh 71813d0fb8 Merge pull request 'Polish: daily snapshot rotation' (#86) from polish-snapshot-rotate into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 10s
2026-07-14 20:12:20 +00:00
Omar Sobh 7a3c04bed7 Polish: daily snapshot rotation script + systemd timer
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 10s
deploy/scripts/rotate-snapshots.sh creates daily-YYYY-MM-DD
snapshot + prunes daily-* older than RETAIN_DAYS. Only touches
its own daily-* namespace so hand-created snapshots (release
anchors etc) never get reaped.

02:00 timer runs ahead of the 03:15 ref-sweep + 03:30 gc so
tonight's fresh snapshot pins protect its blobs from eviction.

DRY_RUN=1 for preview.
2026-07-14 13:12:15 -07:00
osobh f506b446d7 Merge pull request 'Polish: cluster-ref-sweep --apply' (#85) from polish-ref-sweep-apply into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 3s
2026-07-14 20:11:09 +00:00
Omar Sobh 1c5d3c08bd Polish: cluster-ref-sweep --apply forgets stale records
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
Default remains dry-run. --apply iterates the stale set and calls
RefTracking::forget per fp. Blob eviction stays a separate step
(next cluster-gc). Errors are surfaced per-fp, batch continues.
2026-07-14 13:11:04 -07:00
osobh b0b283d897 Merge pull request 'Polish: tests for Phase 6c fixes + RefStore::list' (#84) from polish-store-tests into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 20:09:10 +00:00
Omar Sobh 5dd5d2d00f Polish: tests for Phase 6c TagStore fixes + RefStore::list
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
Retroactive coverage for the layer-split fixes shipped live in
Phase 6c hotfix cycle:
* TagStore::list unions legacy+stamped, dedups by key
* TagStore::get falls through to stamped when legacy absent
* TagStore::delete unlinks both layers + TTL sidecar
* TagStore::contains reports stamped-only pin
* RefStore::list empty + sorted-pairs (stamped wins on collision)

+6 tests. 387 pass.
2026-07-14 13:09:06 -07:00
osobh 7adac2af38 Merge pull request 'Polish: cargo fix' (#83) from polish-cargo-fix into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 20:06:54 +00:00
Omar Sobh 8a502be65c Polish: cargo fix pass (auto-remove unused imports)
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
2026-07-14 13:06:49 -07:00
osobh 88bb5a197f Merge pull request 'Phase 6e hotfix: refs path' (#82) from phase-6e-fix-refs-path into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 12s
2026-07-14 19:44:46 +00:00
Omar Sobh 6d16d30eee Phase 6e hotfix: RefStore lives at <data>/refs-db/ not <data>/
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
Daemon opens refs under <blob_root>/refs-db/ (see services.rs).
FUSE was opening at <data>/ and finding nothing. Match the
daemon's nested convention.
2026-07-14 12:44:41 -07:00
osobh 168309f6ee Merge pull request 'Phase 7i: nightly cluster-ref-sweep timer' (#81) from phase-7i-ref-sweep-timer into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 19:42:18 +00:00
Omar Sobh 4c4b587193 Phase 7i: nightly cluster-ref-sweep systemd timer
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
03:15 local, Persistent=true. Runs ahead of the 03:30 GC so
operators see the stale-fingerprint report before eviction lands.

Environment= drives the Gitea URL; token expected in a drop-in
(clawstor-ref-sweep.service.d/token.conf) so it doesn't sit in
the unit file. README + install recipe updated.
2026-07-14 12:42:02 -07:00
osobh 1ffb08380d Merge pull request 'Phase 7h: nightly cluster-gc systemd timer' (#80) from phase-7h-gc-timer into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 3s
2026-07-14 19:40:37 +00:00
Omar Sobh 74caaed0d1 Phase 7h: nightly cluster-gc systemd timer
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
03:30 local, Persistent=true. Ordered before the Sun 04:00
scrub so scrub reads a fresh post-GC layout.

Default ExecStart is orphan-chunk sweep only (safe on any
node). Fleets that want LRU size-cap eviction add a drop-in:
  systemctl --user edit clawstor-gc.service
  [Service]
  ExecStart=
  ExecStart=%h/clawstor-deploy/claw-store --config ... cluster-gc --evict-to-gb 200

README updated.
2026-07-14 12:40:20 -07:00
osobh d914fd860d Merge pull request 'Phase 7g: weekly cluster-scrub systemd timer' (#79) from phase-7g-scrub-timer into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 10s
2026-07-14 19:39:14 +00:00
Omar Sobh 3000021ac3 Phase 7g: weekly cluster-scrub systemd timer
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
Deploy artifact. Sunday 04:00 local, Persistent=true so a
box that missed the wall-clock moment fires on next boot.

Timer + Service pair — both go under
~/.config/systemd/user/. The service is Type=oneshot; timer
drives it. Non-zero exit from cluster-scrub (integrity issue)
surfaces via systemd failed state; journalctl has the details.

README updated with install recipe.
2026-07-14 12:38:59 -07:00
osobh 7dbf2e9090 Merge pull request 'Phase 6e: expose refs as FUSE files' (#78) from phase-6e-fuse-refs into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 2s
2026-07-14 19:37:38 +00:00
Omar Sobh 7f46e2566b Phase 6e: expose refs as FUSE files
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 2s
<mount>/refs/<fp-hex>   file, content = blob that fp points at

Adds RefStore::list() unioning legacy refs/ and stamped refs-v2/.
FUSE wires the new dir. On lookup, tries get_stamped first then
legacy — matches the resolution order used by claw-cargo build.

Ref files piggy-back the blob inode (same content), so a hex
readable via /blobs/<hex> and /refs/<fp> shares the inode. cheap.
2026-07-14 12:37:20 -07:00
osobh e8e3b05b42 Merge pull request 'Phase 6c fix: hide companion tags from FUSE' (#77) from phase-6c-fuse-hide-companion-tags into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 19:32:36 +00:00
Omar Sobh cf8acadbc1 Phase 6c fix: hide companion tags from FUSE
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 10s
Companion tags store a fingerprint, not a blob-id. Filter them
out of /tags so ls does not show unreadable entries.
2026-07-14 12:32:18 -07:00
osobh bf69938e57 Merge pull request 'Phase 6d hotfix: systemd ExecStart needs %h' (#76) from phase-6d-fix-execstart into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 19:30:57 +00:00
Omar Sobh 6071fdb7d9 Phase 6d hotfix: systemd ExecStart needs %h not ${VAR}
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 12s
status=203/EXEC means the exec path didn't resolve. systemd
expands %h (home) at parse time but does NOT expand ${VAR}
in the ExecStart executable position — only in ExecStart args.
Rewrote to use %h/clawstor-deploy/claw-fuse directly.
2026-07-14 12:30:52 -07:00
osobh ae164863cc Merge pull request 'Phase 6d: systemd user unit for persistent FUSE mount' (#75) from phase-6d-fuse-systemd into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 13s
2026-07-14 19:29:37 +00:00
Omar Sobh b1fc463655 Phase 6d: systemd user unit for persistent FUSE mount
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 15s
Deploy artifact — not built into the binary tree. `deploy/systemd/
clawstor-fuse.service` + README with the install recipe.

Unit shape:
* Type=simple, blocks on the FUSE binary (unmount = SIGTERM).
* ExecStartPre = mkdir -p mount, best-effort lazy unmount of any
  stale prior mount (guards against a hard SIGKILL leaving the
  kernel with a dangling mount).
* Restart=on-failure, RestartSec=5 — recover from transient IO
  errors without operator involvement.
* Environment= for CLAWSTOR_FUSE_BIN / DATA / MOUNT so a
  `systemctl edit` drop-in retargets without editing the unit file.

Dependency chain: clawstor-fuse.service After= + Wants=
clawstor-cluster.service. FUSE reads only the on-disk state so
this is technically not needed for correctness — but keeps the
mount from spinning up on a node where the daemon is broken.
2026-07-14 12:29:32 -07:00
osobh cfd7de8125 Merge pull request 'Phase 6c fix: TagStore delete/contains also see stamped' (#74) from phase-6c-fix-delete-contains into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 10s
2026-07-14 19:27:41 +00:00
Omar Sobh 2c51f0917e Phase 6c fix: TagStore delete()/contains() also see stamped tags
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
Companion to the previous list/get fix. Same layer-split problem:
* delete() only unlinked tags/, leaving stamped tags-v2/ behind.
  Result: `unpin` prints \"no such tag\" for pins created via
  Phase 3c+ RPC even though the tag is right there on disk.
* contains() only checked tags/. Same false-negative.

Fix: both APIs now inspect BOTH layers. delete() unlinks
whichever files exist (either or both) AND removes the TTL
expiry sidecar if present. Returns true when anything was
actually removed.

Legacy behavior preserved: tests unchanged, 381 tests pass.
2026-07-14 12:27:36 -07:00
osobh dfbde85c9f Merge pull request 'Phase 6c fix: TagStore list/get see stamped tags' (#73) from phase-6c-fix-tags-list-stamped into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 12s
2026-07-14 19:25:20 +00:00
Omar Sobh 14a8221e00 Phase 6c fix: TagStore list()/get() see stamped tags
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 12s
Live smoke exposed the gap: `claw-cargo pin` writes to the
stamped store (tags-v2/, Phase 3c+) but `TagStore::list()` +
`get()` only walked the legacy `tags/` layer. Modern pins were
invisible to everything using those APIs — including the new
FUSE tag layer, but also `claw-cargo list-tags`.

Fix:
* list() now unions legacy + stamped entries (deduped by key).
  New private list_stamped_only() walks tags-v2/.
* get() falls through to get_stamped() when the legacy file is
  absent — modern pins resolve without callers knowing which
  layer stored them.

No API breakage: legacy tests still pass unchanged.
381 tests pass.
2026-07-14 12:25:15 -07:00
osobh 5d6e2cd4f4 Merge pull request 'Phase 6c: tags as files in claw-fuse' (#72) from phase-6c-fuse-tags into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 23s
2026-07-14 19:22:20 +00:00
Omar Sobh 3cb32f134a Phase 6c: tags as files in claw-fuse
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
<mount>/tags/<sanitized-name>   ← file, content = current tag's blob

Layers atop the Phase 6a/6b mount. Operator can now `cat` a named
build cache without translating a tag → blob-id first:

  cat <mount>/tags/clawverse:main:latest-cache | tar -tvzf -

Slash → underscore for filenames (tag keys like `a/b:c` land as
`a_b:c`), tags with control chars or NUL are dropped from the
listing. Non-existent value blobs also drop.

Tag file inodes 1_000..9_999. Each getattr / read re-resolves
the tag key (they're mutable — a `pin --replace` under a tag
should show the new blob without unmount).

Reused alloc_blob_ino so tag reads share inodes with /blobs/<hex>
where possible. TagStore opened at `<data_dir>/tags-db` matching
the daemon's convention.
2026-07-14 12:22:15 -07:00
osobh cb0927d04d Merge pull request 'Phase 6b: snapshots as directories in claw-fuse' (#71) from phase-6b-fuse-snapshots into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 10s
2026-07-14 19:18:26 +00:00
Omar Sobh 6f12bd908a Phase 6b: snapshots as directories in claw-fuse
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 18s
Extends the read-only FUSE mount with a `snapshots/` tree:

  <mount>/snapshots/<name>/               ← dir per snapshot
  <mount>/snapshots/<name>/<blob-id-hex>  ← file, content = assembled blob

Lets an operator browse a point-in-time capture by name — `ls`,
`find`, `sha256sum` all just work.

Implementation:
* Inode partitioning: 1 = /, 2 = /blobs, 3 = /snapshots,
  10_000..99_999 = snapshot dirs (lazy allocation), 100_000+
  = blob files (shared with the /blobs tree — same blob has
  the same inode whether reached via /blobs or /snapshots/<n>).
* lookup on /snapshots/<name> validates the snapshot exists via
  SnapshotStore::get.
* lookup on /snapshots/<name>/<blob-hex> validates both that
  the snapshot references that blob AND that the blob is on
  disk — no stale symlinks.
* Reused the shared alloc_blob_ino helper so the /blobs and
  /snapshots trees hand out identical inodes for the same blob.

No new tests: FUSE is integration-heavy and the underlying
snapshot + blob primitives are already covered.
2026-07-14 12:18:22 -07:00
osobh 041a0f5134 Merge pull request 'Phase 6a hotfix: claw-fuse missing mod decls' (#70) from phase-6a-fuse-mod-fix into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 3s
2026-07-14 19:15:36 +00:00
Omar Sobh 35848d6cf7 Phase 6a hotfix: claw-fuse needs full mod list
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
Missed snapshot/sync/zfs mods → build failed on tank with
'unresolved import' errors. Rest of the code was fine.
2026-07-14 12:15:31 -07:00
osobh 22d8bf81a0 Merge pull request 'Phase 6a: read-only FUSE mount over the blob store' (#69) from phase-6a-fuse-mount into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 22s
2026-07-14 19:14:04 +00:00
Omar Sobh c678c08c76 Phase 6a: read-only FUSE mount over the blob store
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
First slice of Phase 6. Ships a minimal, feature-gated `claw-fuse`
binary that mounts the local blob store read-only as a POSIX
filesystem:

  <mount>/blobs/<blob-id-hex>   ← file, content = assembled blob
  <mount>/blobs/                ← dir, ls shows all blob-ids
  <mount>/                      ← dir, contains `blobs`

Lets an operator `tar -tvzf`, `md5sum`, or grep at a cached
tarball without wiring a client. Debug + audit tool for now;
warm-tier git-worktrees + write path come in later slices.

Feature-gated so my macOS dev box doesn't need macFUSE headers
to build the rest of the tree:
* Cargo.toml declares `[[bin]] name = "claw-fuse"` with
  `required-features = ["fuse"]`.
* Feature `fuse` pulls in `fuser = "0.15"`, target-restricted
  to `cfg(target_os = "linux")` — dep resolution never
  considers fuser on other platforms.
* `cargo build` (default) leaves claw-fuse out entirely.
  `cargo build --features fuse --bin claw-fuse` on Linux builds it.

Design notes baked into the impl:
* Inode allocation is lazy — first `lookup` for a hex assigns an
  inode. Avoids pre-indexing the full blob store at mount time
  which would be O(blobs) fs walk before FUSE is even ready.
* getattr / read validate that the manifest exists on every
  call — no stale-inode reads if a blob is GC'd out from under
  us mid-mount. Extra read cost is negligible against the
  per-request FUSE overhead.
* size = manifest.total_size (bytes reported without touching
  chunk files) so `ls -l` is cheap.
* runtime = current-thread tokio, block_on per callback. fuser
  is sync; a full tokio worker pool would just add scheduling
  overhead when callbacks are already serialized by the kernel.

No new tests here — Filesystem impls are integration-heavy and
the underlying BlobStore methods are already covered. The
`fuse` feature build itself will be smoke-tested on tank.

381 tests pass unchanged (feature-gated bin doesn't affect the
existing test surface).
2026-07-14 12:13:59 -07:00
osobh 461f6ac66b Merge pull request 'Phase 8e: cluster-ping migrates to connect_lan_first' (#68) from phase-8e-ping-lan-first into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 19:10:59 +00:00
Omar Sobh cf12554128 Phase 8e: cluster-ping migrates to connect_lan_first
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 12s
Same shape as Phase 8c did for cluster-peer-status + cluster-repair.
New flags: --tailscale-addr (optional) + --lan-probe-ms (default 200).
Route (LAN vs tailnet) printed on the output. Zero flag = identical
to pre-8 single-addr behavior.

Fourth of four operator-facing CLIs now routing-aware
(cluster-peer-status, cluster-repair, cluster-ping done; cluster-ping
was the last outstanding one).

No new tests: pure glue over connect_lan_first, which has its own
unit coverage.
2026-07-14 12:10:54 -07:00
osobh 4a787782f2 Merge pull request 'Phase 7e: claw-cargo smart-clean — 3 local cleanup modes' (#67) from phase-7e-smart-clean into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Failing after 3s
2026-07-14 19:08:48 +00:00
Omar Sobh fa863fdc7d Phase 7e: claw-cargo smart-clean — 3 local cleanup modes
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 3s
Reclaims local target-dir disk in increasing bluntness. All
modes are LOCAL only — the fleet blob cache is untouched, so
`claw-cargo build` after smart-clean restores from peer.

Modes:
* incremental-only — remove target/*/incremental/ across all
  profiles. Safest; keeps final artifacts + deps.
* soft (default) — remove target/ entirely. Blob still on peer.
* hard — soft, but requires --force. Reserved for operators who
  know their build is transient. Rejected without --force even
  in --dry-run so the safety belt can't be trained away.

--dry-run reports paths + byte count without touching disk.

New helpers (unit-tested in isolation):
* find_incremental_dirs(target) — walks target/*/incremental,
  returns only existing entries.
* dir_size_bytes(root) — recursive byte count, silent on read
  errors (used only for reporting, not correctness).

+5 tests: incremental discovery (existing only), missing target
empty, byte sum recursive, missing dir returns 0, hard-without-
force rejects.

381 tests pass (+5). Pre-existing macOS
hot::tests::test_project_target_size_bytes failure unchanged.
2026-07-14 12:08:43 -07:00
osobh eee62b7933 Merge pull request 'Phase 8d: daemon binds a second QuicServer on the tailnet interface' (#66) from phase-8d-tailnet-server-bind into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 19:00:10 +00:00
Omar Sobh dbc1587bcb Phase 8d: daemon binds a second QuicServer on the tailnet interface
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
Live smoke on tank↔architect exposed the gap: bind_rpc_tailscale
was being *advertised* via gossip so peers learned to dial it,
but the daemon never actually LISTENED there. Tailnet dials hit
a closed port.

Fix: when both bind_rpc_lan and bind_rpc_tailscale are set (and
differ), spawn a second QuicServer on the tailnet address. Shares
the same fleet-CA identity + RpcRouter as the LAN listener —
requests from either side hit the same handlers.

If the second bind fails (e.g. tailnet interface not up), we log
a warning and keep the LAN listener alive rather than aborting
daemon startup. Standard graceful-degrade shape.

No new tests here — a live integration test would need two
network interfaces + a running tailscale, which the CI runners
don't have. Coverage happens on the tank+architect deployment:
`ss -lunp` on architect must show TWO clawstor UDP listeners
after this change (10.0.0.13:7702 + 100.104.171.32:7702).

Follow-on: cert SAN for the tailnet address. The current
fleet-CA-signed leaf only has the node name as SAN, so rustls
verification on the client side still checks against
--peer <name> which passes because CN == node name. But a
belt-and-suspenders leaf using fleet-ca-tailscale-sign (Phase
8a) would be more correct.
2026-07-14 12:00:06 -07:00
osobh 1eb2287200 Merge pull request 'Phase 8c hotfix: skip probe deadline when no fallback exists' (#65) from phase-8c-hotfix-lan-only into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 18:45:34 +00:00
Omar Sobh f2c056464c Phase 8c hotfix: skip probe deadline when no fallback exists
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 2s
Live smoke on tank↔architect (both LAN) failed with 200ms probe:
LAN handshake takes longer than that in the wild (TLS 1.3 with
full cert chain + rustls startup on fresh endpoint). The old
single-addr .connect() had no deadline, so pre-8c callers never
noticed.

Fix: when `tailscale` is `None`, treat LAN as unlimited — the
probe deadline only matters as a fall-through trigger, and
there's nothing to fall through to. Callers with a real fallback
addr still get the fast-path routing behavior unchanged.

+1 test (connect_lan_first_lan_only_ignores_probe_deadline)
using a 1-nanosecond probe budget that a real handshake could
never meet — must succeed anyway because no fallback exists.

376 tests pass (+1).
2026-07-14 11:45:29 -07:00
osobh 02adda2f29 Merge pull request 'Phase 8c: cluster-peer-status + cluster-repair support --tailscale-addr' (#64) from phase-8c-cli-tailnet-fallback into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 10s
2026-07-14 18:18:40 +00:00
Omar Sobh 38652c5886 Phase 8c: cluster-peer-status + cluster-repair support --tailscale-addr
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Successful in 11s
Wires the operator CLIs to the Phase 8b connect_lan_first primitive.
Roaming ops (laptop on LTE, coffee-shop wifi) can now pass a
tailnet address alongside the usual --rpc-addr and get the
LAN-first-with-fallback behavior automatically.

New flags on both cluster-peer-status and cluster-repair:
* --tailscale-addr <addr>   — optional tailnet RPC socket. When
                              set, --rpc-addr is tried first with
                              a short deadline, then this on
                              failure/timeout.
* --lan-probe-ms <ms>       — LAN probe deadline. Default 200
                              matches the arch doc.

Zero flag → byte-identical to pre-8c behavior (single-addr dial).
Both flags → chosen route printed in the output header so
operators can see whether LAN or tailnet won.

No new tests: this is thin glue over connect_lan_first, which
already has its own unit coverage. Smoke test live on tank
against architect (LAN), and against fake unroutable + real
tailnet exercises both branches.
2026-07-14 11:18:35 -07:00
osobh acfdf31513 Merge pull request 'Phase 8b: LAN-first probe with tailnet fallback' (#63) from phase-8b-lan-first-probe into main
Build with clawstor cache / Cargo build (clawstor-cached) (push) Successful in 11s
2026-07-14 18:12:24 +00:00
Omar Sobh ef60a7984e Phase 8b: LAN-first probe with tailnet fallback
Build with clawstor cache / Cargo build (clawstor-cached) (pull_request) Failing after 2s
Second Phase 8 slice. Prior transport.connect() took a single
address; the LAN-first-then-Tailscale routing the arch doc calls
out was implicit ("pick lan_addr OR tailscale_addr from gossip
state") and never actually raced or fell through.

New: QuicClient::connect_lan_first(name, lan, tailscale, lan_probe)
* Try LAN first with `lan_probe` deadline (fleet default ~200ms).
* If LAN handshake fails OR the deadline fires → fall back to
  the Tailscale address.
* Both slots None → error immediately (no hang).

Returns (connection, ConnectRoute) so callers + telemetry see
which side won. New enum ConnectRoute::{Lan(addr), Tailscale(addr)}.

+3 tests exercising the three shapes:
- lan-first when LAN reachable (never dials fake tailscale addr)
- fallback when LAN black-holes (240.0.0.1 SYN gets no response;
  probe deadline fires, tailscale server wins)
- errors cleanly when both addrs absent

375 tests pass (+3). Pre-existing macOS
hot::tests::test_project_target_size_bytes failure unchanged.

Follow-ons for Phase 8 completion:
- Wire the peer-connect call sites (RPC forwarding, PeerStatus,
  build-cache) through connect_lan_first with per-peer
  lan/tailscale addrs from gossip state.
- Document the roaming-client config template.
2026-07-14 11:12:19 -07:00