fix(modules): align openclaw mail gateway index host #786

Merged
pdurlej merged 2 commits from codex/issues/openclaw-mail-gateway-index-host into main 2026-06-17 23:01:29 +02:00
Collaborator

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

Canary Context Pack

Product story

Module catalog metadata should agree with the module manifest so agents and operators do not route openclaw-mail-gateway work to the wrong host.

What changed

  • Updated modules/INDEX.yaml for openclaw-mail-gateway from host: rs2000 to host: vps1000.

Why it changed

The new cross-reference lint PR (#783) surfaced this real INDEX/module drift. The manifest already declares the service as a VPS1000 systemd-user runtime proxied through RS2000 Traefik.

Files touched

  • modules/INDEX.yaml

Relevant context

  • modules/openclaw-mail-gateway/module.yaml declares spec.runtime.host: vps1000 and orchestrator: systemd-user.
  • modules/INDEX.yaml previously declared the same module as host: rs2000.

Runtime evidence

No live runtime access. Metadata-only correction.

Known constraints

  • This does not change routing, Traefik, systemd units, DNS, secrets, or runtime state.
  • Public ingress remains documented as RS2000 Traefik proxying to VPS1000.

Explicit out-of-scope

  • No runtime deploy/restart.
  • No exposure/auth change.
  • No broad module index regeneration.

Requested decision

Approve the single-field catalog correction.

Merge blockers

  • If modules/openclaw-mail-gateway/module.yaml is wrong and the service should actually be cataloged as RS2000-owned.

Spec sources read

  • modules/openclaw-mail-gateway/module.yaml for source-of-truth runtime host.
  • modules/INDEX.yaml for index entry.

Verification

  • UV_CACHE_DIR=/Users/pd/Developer/platform/.uv-cache PYTHONPATH=control-plane uv run --project control-plane --extra dev python -m platformctl.cli validate all --json -> exit 0
  • YAML consistency check: {'index_host': 'vps1000', 'module_host': 'vps1000', 'match': True}
  • git diff --check -> passed
Canary status: missing - fire canary 3+3 manually before merge ## Canary Context Pack ### Product story Module catalog metadata should agree with the module manifest so agents and operators do not route `openclaw-mail-gateway` work to the wrong host. ### What changed - Updated `modules/INDEX.yaml` for `openclaw-mail-gateway` from `host: rs2000` to `host: vps1000`. ### Why it changed The new cross-reference lint PR (#783) surfaced this real INDEX/module drift. The manifest already declares the service as a VPS1000 systemd-user runtime proxied through RS2000 Traefik. ### Files touched - `modules/INDEX.yaml` ### Relevant context - `modules/openclaw-mail-gateway/module.yaml` declares `spec.runtime.host: vps1000` and `orchestrator: systemd-user`. - `modules/INDEX.yaml` previously declared the same module as `host: rs2000`. ### Runtime evidence No live runtime access. Metadata-only correction. ### Known constraints - This does not change routing, Traefik, systemd units, DNS, secrets, or runtime state. - Public ingress remains documented as RS2000 Traefik proxying to VPS1000. ### Explicit out-of-scope - No runtime deploy/restart. - No exposure/auth change. - No broad module index regeneration. ### Requested decision Approve the single-field catalog correction. ### Merge blockers - If `modules/openclaw-mail-gateway/module.yaml` is wrong and the service should actually be cataloged as RS2000-owned. ## Spec sources read - `modules/openclaw-mail-gateway/module.yaml` for source-of-truth runtime host. - `modules/INDEX.yaml` for index entry. ## Verification - `UV_CACHE_DIR=/Users/pd/Developer/platform/.uv-cache PYTHONPATH=control-plane uv run --project control-plane --extra dev python -m platformctl.cli validate all --json` -> exit 0 - YAML consistency check: `{'index_host': 'vps1000', 'module_host': 'vps1000', 'match': True}` - `git diff --check` -> passed
fix(modules): align openclaw mail gateway index host
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-client-dry-run / dry-run (pull_request) Waiting to run
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
patchwarden-pr-sanity / sanity (pull_request) Waiting to run
platformctl plan / auto-apply scope (pull_request) Successful in 20s
canary-required / canary (pull_request) Has been cancelled
9f1adefb17
Merge remote-tracking branch 'origin/main' into codex/issues/openclaw-mail-gateway-index-host-refresh
Some checks failed
canary-required / collect-diff (pull_request) Successful in 3s
platformctl plan / auto-apply scope (pull_request) Successful in 20s
canary-required / canary (pull_request) Successful in 13s
base-is-main / guard (pull_request) Successful in 1s
patchwarden-pr-sanity / collect-diff (pull_request) Successful in 4s
patchwarden-client-dry-run / dry-run (pull_request) Failing after 19s
patchwarden-client-dry-run / collect-diff (pull_request) Successful in 3s
patchwarden-pr-sanity / sanity (pull_request) Failing after 1m55s
b41a2fa52c
First-time contributor

Patchwarden PR sanity

  • Status: eligible_sanity_clean
  • PR: 786
  • Commit: b41a2fa52c0dc7f7c63abffb25fa77393485474a
  • Security-sensitive label: missing
  • 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

  • medium Missing canary before merge

    • Evidence: PR description states 'Canary status: missing - fire canary 3+3 manually before merge'. The change is metadata-only, but if canary is a required safety step, merging without it violates policy.
    • Next: Fire canary 3+3 as indicated before merging.

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-786 --> # Patchwarden PR sanity - Status: `eligible_sanity_clean` - PR: `786` - Commit: `b41a2fa52c0dc7f7c63abffb25fa77393485474a` - Security-sensitive label: `missing` - 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` - **`medium`** Missing canary before merge - Evidence: `PR description states 'Canary status: missing - fire canary 3+3 manually before merge'. The change is metadata-only, but if canary is a required safety step, merging without it violates policy.` - Next: Fire canary 3+3 as indicated before merging. ### `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.
Author
Collaborator

Role: executor

PR-zero verification before merge:

  • modules/openclaw-mail-gateway/module.yaml and runbook both identify host vps1000; PR fixes modules/INDEX.yaml drift from rs2000 to vps1000.
  • git diff --check origin/main...HEAD -> clean.
  • UV_CACHE_DIR=/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json -> exit 0.

No runtime mutation; catalog-only correction.

**Role:** executor PR-zero verification before merge: - `modules/openclaw-mail-gateway/module.yaml` and runbook both identify host `vps1000`; PR fixes `modules/INDEX.yaml` drift from `rs2000` to `vps1000`. - `git diff --check origin/main...HEAD` -> clean. - `UV_CACHE_DIR=/tmp/platform-uv-cache PYTHONPATH=control-plane uv run --project control-plane python -m platformctl.cli validate all --json` -> exit 0. No runtime mutation; catalog-only correction.
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
3 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!786
No description provided.