docs(vault): define accelerated sunset path #612

Merged
pdurlej merged 1 commit from codex/m04-vault-accelerated-sunset-plan into main 2026-05-29 21:55:15 +02:00
Collaborator

Summary

Defines the accelerated M04 Vault sunset path after inventory showed Vault is not a general secret backend and only backs safe-session SSH signing.

What changed

  • Amends ADR-0024: Infisical remains the only canonical secret backend for RS2000/VPS1000.
  • Supersedes the old bulk Vault-to-Infisical mirror path unless future KV evidence appears.
  • Records the observed safe-session Vault SSH role policy.
  • Defines required gates for local OpenSSH CA signer, CA key handling, smoke/rollback, audit, rotation, and destructive cleanup.

DeepSeek v4 Pro redteam

DeepSeek returned HOLD twice and the findings were folded into the docs:

  • role policy parity
  • CA key handling/logs/core dumps/swap
  • no new signer API/socket/route
  • audit metadata
  • TTL/not_before/options smoke
  • rollback and rotation gates

Validation

  • platformctl validate all --json: pass (exitCode=0, 88 modules)

Non-goals

  • No runtime mutation.
  • No Vault stop/delete.
  • No secret values.
  • No new Infisical writes.
  • No safe-session-api code changes yet.

Closes #609.

## Summary Defines the accelerated M04 Vault sunset path after inventory showed Vault is not a general secret backend and only backs safe-session SSH signing. ## What changed - Amends ADR-0024: Infisical remains the only canonical secret backend for RS2000/VPS1000. - Supersedes the old bulk Vault-to-Infisical mirror path unless future KV evidence appears. - Records the observed safe-session Vault SSH role policy. - Defines required gates for local OpenSSH CA signer, CA key handling, smoke/rollback, audit, rotation, and destructive cleanup. ## DeepSeek v4 Pro redteam DeepSeek returned HOLD twice and the findings were folded into the docs: - role policy parity - CA key handling/logs/core dumps/swap - no new signer API/socket/route - audit metadata - TTL/not_before/options smoke - rollback and rotation gates ## Validation - `platformctl validate all --json`: pass (`exitCode=0`, 88 modules) ## Non-goals - No runtime mutation. - No Vault stop/delete. - No secret values. - No new Infisical writes. - No `safe-session-api` code changes yet. Closes #609.
docs(vault): define accelerated sunset path
Some checks failed
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Successful in 14s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 24s
base-is-main / guard (pull_request) Successful in 1s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / sanity (pull_request) Failing after 3m44s
6e999d6461
Record the 2026-05-29 Vault inventory conclusion: Vault is not a peer secret backend and has no observed KV store to mirror. Define the safe-session local SSH CA signer path, required policy parity, key-handling gates, smoke/rollback evidence, audit expectations, and destructive cleanup boundary.
codex force-pushed codex/m04-vault-accelerated-sunset-plan from 6e999d6461
Some checks failed
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Successful in 14s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 24s
base-is-main / guard (pull_request) Successful in 1s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / sanity (pull_request) Failing after 3m44s
to 62ac1e7585
Some checks failed
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 6s
canary-required / canary (pull_request) Successful in 15s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 22s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m50s
2026-05-29 21:19:17 +02:00
Compare
codex force-pushed codex/m04-vault-accelerated-sunset-plan from 62ac1e7585
Some checks failed
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 6s
canary-required / canary (pull_request) Successful in 15s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 22s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m50s
to add156665a
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 5s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 5s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 5s
canary-required / canary (pull_request) Successful in 14s
patchwarden-client-dry-run / dry-run (pull_request) Successful in 23s
patchwarden-pr-sanity / sanity (pull_request) Successful in 2m4s
2026-05-29 21:44:52 +02:00
Compare
Author
Collaborator

Patchwarden PR sanity

  • Status: advisory_findings
  • PR: 612
  • Commit: add156665aea2ec6cd908a32374dc95d7424ad96
  • Security-sensitive label: present
  • Authority: advisory model review plus deterministic blockers only
  • 3+3 canary: still alive; this does not replace it

Deterministic findings

  • info sensitive-path-touched Sensitive path touched — decisions/0024-infisical-primary-secrets-pipeline.md
    • Evidence: decisions/0024-infisical-primary-secrets-pipeline.md
    • Next: Route through the existing 3+3/risk-tier process; model review remains advisory.

Model reviewers

global-glm / glm-5.1:cloud

  • Status: ok
  • Verdict: OK
  • Findings: none

global-deepseek / deepseek-v4-pro:cloud

  • Status: ok
  • Verdict: OK
  • Findings: none

redteam / kimi-k2.6:cloud

  • Status: ok

  • Verdict: NOT_OK

  • high Contradictory CA key ownership: root-only artifact vs non-root API

    • Evidence: migrations/vault-to-infisical.md (+3) requires the CA private key be stored in a 'root-only runtime artifact', while decisions/0024-infisical-primary-secrets-pipeline.md and state/reports/m04-vault-accelerated-sunset-decision-2026-05-29.md
    • Next: Align the migration doc to require service-user ownership (0600) or explicitly document the privilege-separation mechanism that allows a non-root API to consume a root-protected key.

Policy notes

  • GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot.
  • Optional red-team model is enabled only when PLATFORMCTL_PR_SANITY_REDTEAM_MODEL is configured.
  • Auto-merge is not enabled here.
<!-- patchwarden-pr-sanity:pdurlej/platform:PR-612 --> # Patchwarden PR sanity - Status: `advisory_findings` - PR: `612` - Commit: `add156665aea2ec6cd908a32374dc95d7424ad96` - Security-sensitive label: `present` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings - **`info` `sensitive-path-touched`** Sensitive path touched — `decisions/0024-infisical-primary-secrets-pipeline.md` - Evidence: `decisions/0024-infisical-primary-secrets-pipeline.md` - Next: Route through the existing 3+3/risk-tier process; model review remains advisory. ## Model reviewers ### `global-glm` / `glm-5.1:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `global-deepseek` / `deepseek-v4-pro:cloud` - Status: `ok` - Verdict: `OK` - Findings: none ### `redteam` / `kimi-k2.6:cloud` - Status: `ok` - Verdict: `NOT_OK` - **`high`** Contradictory CA key ownership: root-only artifact vs non-root API - Evidence: `migrations/vault-to-infisical.md (+3) requires the CA private key be stored in a 'root-only runtime artifact', while decisions/0024-infisical-primary-secrets-pipeline.md and state/reports/m04-vault-accelerated-sunset-decision-2026-05-29.md ` - Next: Align the migration doc to require service-user ownership (0600) or explicitly document the privilege-separation mechanism that allows a non-root API to consume a root-protected key. ## Policy notes - GLM 5.1 + DeepSeek V4 Pro are the operator-required model mix for this bot. - Optional red-team model is enabled only when `PLATFORMCTL_PR_SANITY_REDTEAM_MODEL` is configured. - Auto-merge is not enabled here.
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 milestone
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!612
No description provided.