fix(cutover): move runtime defaults off legacy root #579

Merged
pdurlej merged 1 commit from codex/m01-runtime-defaults-off-legacy into main 2026-05-28 23:28:46 +02:00
Collaborator

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

Summary

Moves repo-side M01 runtime defaults off /opt/vps-home-platform-infra without mutating RS2000 runtime state.

This is the first post-audit closeout step after Wave 1: the read-only RS2000 legacy audit reported blocker references in active Compose defaults and in platformctl apply's default remote root. This PR removes those live defaults while preserving safety guard references as review-only audit findings.

What changed

  • platformctl apply now defaults to /opt/pdurlej-platform/current instead of the old legacy root.
  • Compose defaults now point env/config/data mounts to /opt/pdurlej-platform/runtime/legacy-import/....
  • control-plane/platformctl/safety.py remains allowed to mention the legacy root as a sacred-path guardrail, but the audit classifies it as review-only rather than a live blocker.
  • Tests were updated for the new default root and the audit classification.

Runtime precondition

Before the first real deploy/apply using these defaults, RS2000 must have the conservative copy in place:

  • /opt/pdurlej-platform/runtime/legacy-import/env
  • /opt/pdurlej-platform/runtime/legacy-import/config/integrations
  • /opt/pdurlej-platform/runtime/legacy-import/data/integrations

If runtime env overrides are already set, they may continue to override these defaults. This PR does not copy data, restart containers, or delete old paths.

Safety

  • No runtime mutation.
  • No deletion.
  • No restart.
  • No secret values printed or committed.
  • Old /opt/vps-home-platform-infra data/env paths remain protected by safety.py.
  • Destructive cleanup remains blocked by m01-destructive-cleanup-approved.

Validation

  • UV_CACHE_DIR=/private/tmp/platform-uv-cache uv run pytest platformctl/tests/test_rs2000_legacy_audit.py platformctl/tests/test_apply_phase3.py platformctl/tests/test_apply_env_file.py => 68 passed
  • UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli cutover rs2000-legacy-audit --json => exit 0; 0 blockers, 0 unknown, 0 commandErrors
  • 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
  • DeepSeek V4 Pro review => OK; noted the runtime precondition above.

Spec sources read

  • state/cutover/rs2000-live-data-config-migration.md
  • state/cutover/rs2000-post-soak-legacy-cleanup.md
  • decisions/0020-post-soak-legacy-cleanup-and-platform-modularization.md
  • compose/apps/compose.yaml
  • compose/core/compose.yaml
  • control-plane/platformctl/apply.py
  • control-plane/platformctl/cutover/rs2000_legacy_audit.py

Non-goals

  • No data copy.
  • No service remount.
  • No Docker restart.
  • No runtime smoke claim.
  • No quarantine/tombstone.
  • No destructive cleanup.
Canary status: missing - fire canary 3+3 manually before merge ## Summary Moves repo-side M01 runtime defaults off `/opt/vps-home-platform-infra` without mutating RS2000 runtime state. This is the first post-audit closeout step after Wave 1: the read-only RS2000 legacy audit reported blocker references in active Compose defaults and in `platformctl apply`'s default remote root. This PR removes those live defaults while preserving safety guard references as review-only audit findings. ## What changed - `platformctl apply` now defaults to `/opt/pdurlej-platform/current` instead of the old legacy root. - Compose defaults now point env/config/data mounts to `/opt/pdurlej-platform/runtime/legacy-import/...`. - `control-plane/platformctl/safety.py` remains allowed to mention the legacy root as a sacred-path guardrail, but the audit classifies it as review-only rather than a live blocker. - Tests were updated for the new default root and the audit classification. ## Runtime precondition Before the first real deploy/apply using these defaults, RS2000 must have the conservative copy in place: - `/opt/pdurlej-platform/runtime/legacy-import/env` - `/opt/pdurlej-platform/runtime/legacy-import/config/integrations` - `/opt/pdurlej-platform/runtime/legacy-import/data/integrations` If runtime env overrides are already set, they may continue to override these defaults. This PR does not copy data, restart containers, or delete old paths. ## Safety - No runtime mutation. - No deletion. - No restart. - No secret values printed or committed. - Old `/opt/vps-home-platform-infra` data/env paths remain protected by `safety.py`. - Destructive cleanup remains blocked by `m01-destructive-cleanup-approved`. ## Validation - `UV_CACHE_DIR=/private/tmp/platform-uv-cache uv run pytest platformctl/tests/test_rs2000_legacy_audit.py platformctl/tests/test_apply_phase3.py platformctl/tests/test_apply_env_file.py` => 68 passed - `UV_CACHE_DIR=/private/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli cutover rs2000-legacy-audit --json` => exit 0; `0` blockers, `0` unknown, `0` commandErrors - `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 - DeepSeek V4 Pro review => OK; noted the runtime precondition above. ## Spec sources read - `state/cutover/rs2000-live-data-config-migration.md` - `state/cutover/rs2000-post-soak-legacy-cleanup.md` - `decisions/0020-post-soak-legacy-cleanup-and-platform-modularization.md` - `compose/apps/compose.yaml` - `compose/core/compose.yaml` - `control-plane/platformctl/apply.py` - `control-plane/platformctl/cutover/rs2000_legacy_audit.py` ## Non-goals - No data copy. - No service remount. - No Docker restart. - No runtime smoke claim. - No quarantine/tombstone. - No destructive cleanup.
fix(cutover): move runtime defaults off legacy root
Some checks failed
canary-required / collect-diff (pull_request) Successful in 4s
platformctl plan / auto-apply scope (pull_request) Successful in 23s
pyfallow / Pyfallow gate (control-plane) (pull_request) Successful in 20s
python-ci / Python 3.12 (pull_request) Successful in 42s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s
python-ci / Python 3.11 (pull_request) Successful in 42s
python-ci / Python 3.13 (pull_request) Successful in 42s
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 21s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m35s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
e33033fbdc
pdurlej deleted branch codex/m01-runtime-defaults-off-legacy 2026-05-28 23:28:47 +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!579
No description provided.