docs(vault): record Phase 1 inventory result #610

Merged
pdurlej merged 2 commits from codex/m04-vault-inventory-blocker into main 2026-05-30 03:13:11 +02:00
Collaborator

Summary

Records the #607 read-only Vault Phase 1 inventory result.

The live rs2000 HashiCorp Vault is not a bulk KV secret store ready for mirror-to-Infisical. The metadata-only inventory shows Vault is currently shaped as the internal safe-session SSH signer:

  • secret engines: cubbyhole/, identity/, ssh/, sys/
  • policy: safe-session-signer
  • AppRole: safe-session-api
  • SSH role: safe-session
  • no observed secret/ KV engine

What changed

  • Added state/reports/m04-vault-phase1-inventory-2026-05-29.md.
  • Recorded sanitized evidence only: engine names, policy names, role names, module cross-check.
  • Recommended no-go for Phase 2 Vault-to-Infisical mirror unless a later inventory finds actual KV secrets.
  • Created follow-up #609 for deciding the safe-session Vault signer future before continuing Vault sunset.

Safety

  • No Vault values printed.
  • No vault kv get -field=<key> used.
  • No Infisical writes.
  • No Vault seal/unseal/restart/stop.
  • No volume mutation.
  • No cleanup command.

Validation

  • PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json initially hit sandbox cache permissions under ~/.cache/uv.
  • Re-run with local cache passed: UV_CACHE_DIR=/Users/pd/Developer/iskra-platform-2026-04-30/.uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json -> exitCode 0, all modules OK.

Closes #607.

## Summary Records the #607 read-only Vault Phase 1 inventory result. The live `rs2000` HashiCorp Vault is not a bulk KV secret store ready for mirror-to-Infisical. The metadata-only inventory shows Vault is currently shaped as the internal `safe-session` SSH signer: - secret engines: `cubbyhole/`, `identity/`, `ssh/`, `sys/` - policy: `safe-session-signer` - AppRole: `safe-session-api` - SSH role: `safe-session` - no observed `secret/` KV engine ## What changed - Added `state/reports/m04-vault-phase1-inventory-2026-05-29.md`. - Recorded sanitized evidence only: engine names, policy names, role names, module cross-check. - Recommended no-go for Phase 2 Vault-to-Infisical mirror unless a later inventory finds actual KV secrets. - Created follow-up #609 for deciding the `safe-session` Vault signer future before continuing Vault sunset. ## Safety - No Vault values printed. - No `vault kv get -field=<key>` used. - No Infisical writes. - No Vault seal/unseal/restart/stop. - No volume mutation. - No cleanup command. ## Validation - `PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` initially hit sandbox cache permissions under `~/.cache/uv`. - Re-run with local cache passed: `UV_CACHE_DIR=/Users/pd/Developer/iskra-platform-2026-04-30/.uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` -> exitCode 0, all modules OK. Closes #607.
docs(vault): record phase 1 inventory
Some checks failed
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Failing after 1m0s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m31s
b7531681eb
codex force-pushed codex/m04-vault-inventory-blocker from b7531681eb
Some checks failed
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Failing after 1m0s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m31s
to cfb96779ef
Some checks failed
canary-required / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Failing after 54s
base-is-main / guard (pull_request) Successful in 1s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
patchwarden-pr-sanity / sanity (pull_request) Successful in 1m54s
2026-05-30 02:39:48 +02:00
Compare
codex added this to the 04 - Vault sunset milestone 2026-05-30 02:40:05 +02:00
Author
Collaborator

Patchwarden PR sanity

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

Deterministic findings

No deterministic findings.

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: OK
  • Findings: none

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-610 --> # Patchwarden PR sanity - Status: `eligible_sanity_clean` - PR: `610` - Commit: `fac33523e42a3ebce2e5002c2fb4093819427b35` - Security-sensitive label: `present` - Authority: advisory model review plus deterministic blockers only - 3+3 canary: still alive; this does not replace it ## Deterministic findings No deterministic findings. ## 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: `OK` - Findings: none ## 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.
codex force-pushed codex/m04-vault-inventory-blocker from cfb96779ef
Some checks failed
canary-required / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 4s
canary-required / canary (pull_request) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Failing after 54s
base-is-main / guard (pull_request) Successful in 1s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
patchwarden-pr-sanity / sanity (pull_request) Successful in 1m54s
to fac33523e4
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
canary-required / collect-diff (pull_request) Successful in 4s
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) Has been skipped
patchwarden-client-dry-run / dry-run (pull_request) Successful in 52s
patchwarden-pr-sanity / sanity (pull_request) Successful in 1m24s
2026-05-30 03:07:54 +02:00
Compare
pdurlej approved these changes 2026-05-30 03:13:10 +02:00
pdurlej left a comment

Operator-delegated approval for nondestructive M04 Phase 1 inventory evidence. All Forgejo checks are green on expected head; local validate all and diff check passed; this records last-dependency evidence only and does not mutate runtime.

Operator-delegated approval for nondestructive M04 Phase 1 inventory evidence. All Forgejo checks are green on expected head; local validate all and diff check passed; this records last-dependency evidence only and does not mutate runtime.
pdurlej referenced this pull request from a commit 2026-05-30 03:13:13 +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 milestone
No project
No assignees
2 participants
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!610
No description provided.