docs(secrets): record W4a token auth evidence #436

Merged
pdurlej merged 1 commit from codex/w4a/deploy-runner-token-auth-evidence into main 2026-05-24 19:07:36 +02:00
Collaborator

Canary status: missing - rely on required Forgejo checks before merge

Canary Context Pack

Product story

W4a needs to close the temporary deploy-runner direct PAT state without turning credential cleanup into guesswork. This PR records the read-only evidence before any runtime mutation.

What changed

  • Adds state/reports/w4a-deploy-runner-token-auth-2026-05-24.md.
  • Updates state/cycle/W4-secrets-access-hardening-output.md with W4a status.
  • Updates state/STATUS_NOW.md with the removal gate and evidence summary.

Why it changed

The runner is using Infisical Token Auth in real auto-apply logs, but the direct fallback key PLATFORMCTL_FORGEJO_TOKEN is still present. That should be removed only with explicit operator approval and rollback.

Files touched

  • state/reports/w4a-deploy-runner-token-auth-2026-05-24.md
  • state/cycle/W4-secrets-access-hardening-output.md
  • state/STATUS_NOW.md

Relevant context

  • #265 deploy-runner direct PAT to Infisical.
  • #274 Token Auth expiry rotation.
  • docs/ci/runner-contract.md target state: no runner-local direct PAT as normal state.
  • control-plane/platformctl/apply.py token resolution order.

Runtime evidence

Read-only only:

  • Token Auth file exists with mode 600, owner forgejo-deploy:forgejo-deploy, size 333 bytes.
  • Runner env file exists with mode 600, owner forgejo-deploy:forgejo-deploy, size 907 bytes.
  • Direct fallback key still present: PLATFORMCTL_FORGEJO_TOKEN.
  • Last 48h auto-apply logs checked: 16.
  • forgejo_token_source=infisical-token-auth: 16 logs.
  • forgejo_token_source=direct-env-fallback: 0 logs.
  • Watchdog positive restarts in last 24h: 0.

Known constraints

  • No secret value was read or printed.
  • Direct fallback removal is not part of this PR.
  • Removing the fallback is a runtime credential mutation and requires w4a-remove-direct-pat-approved.

Explicit out-of-scope

  • Editing RS2000 runtime files.
  • Restarting the deploy runner.
  • Running a new auto-apply smoke.
  • Closing #265.

Requested decision

Approve merge if checks pass. After merge, operator can approve w4a-remove-direct-pat-approved to remove the dead fallback with backup + rollback.

Merge blockers

  • Any secret value in the report.
  • Any statement that direct fallback is removed already.
  • Any weakening of the explicit removal gate.

Spec sources read

  • state/cycle/W4-secrets-access-hardening-output.md
  • state/STATUS_NOW.md
  • docs/ci/runner-contract.md
  • runbooks/forgejo-actions-runner.md
  • control-plane/platformctl/apply.py
  • RS2000 read-only systemd/stat/journal/action-log metadata

Refs #265, #274.

Canary status: missing - rely on required Forgejo checks before merge ## Canary Context Pack ### Product story W4a needs to close the temporary deploy-runner direct PAT state without turning credential cleanup into guesswork. This PR records the read-only evidence before any runtime mutation. ### What changed - Adds `state/reports/w4a-deploy-runner-token-auth-2026-05-24.md`. - Updates `state/cycle/W4-secrets-access-hardening-output.md` with W4a status. - Updates `state/STATUS_NOW.md` with the removal gate and evidence summary. ### Why it changed The runner is using Infisical Token Auth in real auto-apply logs, but the direct fallback key `PLATFORMCTL_FORGEJO_TOKEN` is still present. That should be removed only with explicit operator approval and rollback. ### Files touched - `state/reports/w4a-deploy-runner-token-auth-2026-05-24.md` - `state/cycle/W4-secrets-access-hardening-output.md` - `state/STATUS_NOW.md` ### Relevant context - #265 deploy-runner direct PAT to Infisical. - #274 Token Auth expiry rotation. - `docs/ci/runner-contract.md` target state: no runner-local direct PAT as normal state. - `control-plane/platformctl/apply.py` token resolution order. ### Runtime evidence Read-only only: - Token Auth file exists with mode `600`, owner `forgejo-deploy:forgejo-deploy`, size `333` bytes. - Runner env file exists with mode `600`, owner `forgejo-deploy:forgejo-deploy`, size `907` bytes. - Direct fallback key still present: `PLATFORMCTL_FORGEJO_TOKEN`. - Last 48h auto-apply logs checked: 16. - `forgejo_token_source=infisical-token-auth`: 16 logs. - `forgejo_token_source=direct-env-fallback`: 0 logs. - Watchdog positive restarts in last 24h: 0. ### Known constraints - No secret value was read or printed. - Direct fallback removal is not part of this PR. - Removing the fallback is a runtime credential mutation and requires `w4a-remove-direct-pat-approved`. ### Explicit out-of-scope - Editing RS2000 runtime files. - Restarting the deploy runner. - Running a new auto-apply smoke. - Closing #265. ### Requested decision Approve merge if checks pass. After merge, operator can approve `w4a-remove-direct-pat-approved` to remove the dead fallback with backup + rollback. ### Merge blockers - Any secret value in the report. - Any statement that direct fallback is removed already. - Any weakening of the explicit removal gate. ## Spec sources read - `state/cycle/W4-secrets-access-hardening-output.md` - `state/STATUS_NOW.md` - `docs/ci/runner-contract.md` - `runbooks/forgejo-actions-runner.md` - `control-plane/platformctl/apply.py` - RS2000 read-only systemd/stat/journal/action-log metadata Refs #265, #274.
docs(secrets): record W4a token auth evidence
All checks were successful
base-is-main / guard (pull_request) Successful in 1s
canary-required / 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-pr-sanity / sanity (pull_request) Successful in 23s
c0810d6b2b
pdurlej approved these changes 2026-05-24 19:06:56 +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!436
No description provided.