fix(cutover): support legacy import without rsync #582

Merged
pdurlej merged 1 commit from codex/m01-legacy-import-tar-fallback into main 2026-05-28 23:50:00 +02:00
Collaborator

Canary status: missing - fire canary 3+3 manually before merge

Summary

Fixes the M01 legacy-import helper after the first RS2000 dry-run found that rsync is not installed on the host.

The helper now prefers rsync when available and falls back to tar otherwise. The fallback preserves ownership, modes, symlinks, and timestamps, but does not claim ACL/xattr parity.

Evidence

Read-only/dry-run on RS2000 before this fix failed with:

rsync: command not found

Running the patched helper over SSH in dry-run mode for config/ntfy/server.yml succeeded with tool=tar and emitted only metadata:

DRY-RUN config/ntfy/server.yml tool=tar
Source apparent size: 4.0K

No file contents or secret values were printed.

What changed

  • Added copy tool detection: rsync first, tar fallback.
  • Added tar dry-run metadata mode that avoids file listing.
  • Added tar apply mode for hosts without rsync.
  • Documented the fallback behavior.

Safety

  • Still dry-run by default.
  • Still requires --yes for writes.
  • Still no delete, rename, restart, chmod sweep, or secret printing.
  • Tar fallback does not use verbose/list mode.
  • Roots remain hard-gated to M01 source/target paths.

Validation

  • bash -n scripts/cutover/prepare-legacy-import.sh => pass
  • Local dry-run for missing config/ntfy/server.yml => safe SKIP
  • RS2000 dry-run via SSH for config/ntfy/server.yml using patched script => pass, tool=tar
  • UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json => exit 0
  • git diff --check => pass

Non-goals

  • Does not run live copy.
  • Does not update Compose mounts.
  • Does not restart services.
  • Does not delete or quarantine legacy paths.
Canary status: missing - fire canary 3+3 manually before merge ## Summary Fixes the M01 legacy-import helper after the first RS2000 dry-run found that `rsync` is not installed on the host. The helper now prefers `rsync` when available and falls back to `tar` otherwise. The fallback preserves ownership, modes, symlinks, and timestamps, but does not claim ACL/xattr parity. ## Evidence Read-only/dry-run on RS2000 before this fix failed with: ```text rsync: command not found ``` Running the patched helper over SSH in dry-run mode for `config/ntfy/server.yml` succeeded with `tool=tar` and emitted only metadata: ```text DRY-RUN config/ntfy/server.yml tool=tar Source apparent size: 4.0K ``` No file contents or secret values were printed. ## What changed - Added copy tool detection: `rsync` first, `tar` fallback. - Added tar dry-run metadata mode that avoids file listing. - Added tar apply mode for hosts without `rsync`. - Documented the fallback behavior. ## Safety - Still dry-run by default. - Still requires `--yes` for writes. - Still no delete, rename, restart, chmod sweep, or secret printing. - Tar fallback does not use verbose/list mode. - Roots remain hard-gated to M01 source/target paths. ## Validation - `bash -n scripts/cutover/prepare-legacy-import.sh` => pass - Local dry-run for missing `config/ntfy/server.yml` => safe SKIP - RS2000 dry-run via SSH for `config/ntfy/server.yml` using patched script => pass, `tool=tar` - `UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` => exit 0 - `git diff --check` => pass ## Non-goals - Does not run live copy. - Does not update Compose mounts. - Does not restart services. - Does not delete or quarantine legacy paths.
fix(cutover): support legacy import without rsync
Some checks failed
canary-required / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
base-is-main / guard (pull_request) Successful in 1s
canary-required / canary (pull_request) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Successful in 20s
patchwarden-pr-sanity / sanity (pull_request) Failing after 49s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
55394ab90c
pdurlej deleted branch codex/m01-legacy-import-tar-fallback 2026-05-28 23:50:00 +02:00
Sign in to join this conversation.
No reviewers
No labels
W6d-automerge-calibration
agent/claude-code
agent/codex
agent/hermes
agent/iskra
agent/ollama
agent/patchwarden
automerge-candidate
class/security-sensitive
cutover-gate
dependency/blocked
dependency/blocks-others
dependency/cross-repo
dependency/needs-confirmation
domain:agents
domain:ci
domain:docs
domain:forgejo
domain:infra
domain:memory
domain:runtime
domain:signal
domain:ux
flow/architecture
flow/blocked
flow/deployed
flow/done
flow/implementation
flow/intake
flow/maintained
flow/observed
flow/ready
flow/refining
flow/retired
flow/review
iterating
judge/codex-candidate
judge/hermes-candidate
judge/low-confidence
judge/needs-refinement
judge/operator-needed
judge/p0
judge/p1
judge/p2
judge/p3
judge/park
judge/patchwarden-candidate
judge/stale-priority
kind/adr
kind/bug
kind/chore
kind/feature
kind/infra
kind/ops
kind/refactor
kind/research
large-impact
merge/auto
merge/manual
merge/manual-dependency-conflict
merge/manual-failing-tests
merge/manual-merge-conflict
merge/manual-missing-review
merge/manual-operator-preference
merge/manual-red-zone
merge/manual-security-sensitive
merge/manual-unclear-scope
merge/manual-unknown
meta
mode:operator-only
mode:patchwarden-iskra-approved
mode:safe-auto
needs-operator-decision
needs-triage
not-ready
observed/erroring
observed/needs-followup
observed/pending
observed/retire-candidate
observed/unused
observed/used
operator-emotional
owner-attention
phase/02
phase/03
priority:p0
priority:p1
priority:p2
priority:p3
proposed
ready-for-agent
ready-for-operator
recovery
review:claude-reviewed
review:codex-reviewed
review:dziadek-reviewed
review:needs-human
risk/exposure
risk/process
risk/product
risk/runtime
safety:external-write
safety:no-prod-mutation
safety:prod-impact
safety:secret-touch
size/large
size/medium
size/small
size/tiny
size/unknown
source/adr
source/agent-generated
source/manual
source/operator-chat
source/voice-note
status:blocked
status:codex-ready
status:merged:pending-evidence
status:needs-evidence
status:operator-needed
status:parked
tier/full
tier/lite
tier/stacked
tier:0-platform-substrate
tier:1-iskra-value-layer
tier:2-tools-products-modules
type:bug
type:chore
type:docs
type:feat
type:policy
type:research
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
pdurlej/platform!582
No description provided.