audit(healthchecks): inventory which modules lack healthcheck configuration #765

Closed
opened 2026-06-08 23:09:24 +02:00 by ollama · 1 comment
Collaborator

Spec sources (whitelist)

  • modules/*/module.yaml — scan spec.health sections for healthcheck configuration
  • schema/module.schema.json — healthcheck schema definition
  • 2-3 reference modules with good healthcheck config (e.g., traefik, honcho-api)

Extracted context

Not all 88 modules have healthcheck configurations. Docker needs healthcheck to auto-restart failing containers and report status accurately. Without healthcheck, a container can be "up" but actually failing.

Scope

  1. Scan all 88 modules/*/module.yaml files for:
    • spec.health.compose.required_healthy — docker-compose healthcheck
    • spec.health.http — HTTP healthcheck
    • Absence of both = gap
  2. Classify findings:
    • HAS_HEALTHCHECK: has at least one healthcheck method
    • NO_HEALTHCHECK: has no healthcheck configuration
    • NOT_APPLICABLE: external/orchestrator not docker-compose
  3. For each NO_HEALTHCHECK gap, create a follow-up issue with the module name and recommended healthcheck type

Acceptance criteria

  • Complete healthcheck coverage inventory
  • List of modules missing healthchecks
  • Follow-up issues created for each gap (or one batch issue if many gaps)

Do NOT read

  • Read module.yaml files only for health section
  • Do not deep-read full manifests

Agent notes

  • Recommended executor: Gemini 3.5 Flash (pure investigation)
  • Size: Trivial (0 LOC — investigation + report)
  • Review tier: tier/trivial
  • Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §DR1
## Spec sources (whitelist) - `modules/*/module.yaml` — scan `spec.health` sections for healthcheck configuration - `schema/module.schema.json` — healthcheck schema definition - 2-3 reference modules with good healthcheck config (e.g., traefik, honcho-api) ## Extracted context Not all 88 modules have healthcheck configurations. Docker needs healthcheck to auto-restart failing containers and report status accurately. Without healthcheck, a container can be "up" but actually failing. ## Scope 1. Scan all 88 `modules/*/module.yaml` files for: - `spec.health.compose.required_healthy` — docker-compose healthcheck - `spec.health.http` — HTTP healthcheck - Absence of both = gap 2. Classify findings: - HAS_HEALTHCHECK: has at least one healthcheck method - NO_HEALTHCHECK: has no healthcheck configuration - NOT_APPLICABLE: external/orchestrator not docker-compose 3. For each NO_HEALTHCHECK gap, create a follow-up issue with the module name and recommended healthcheck type ## Acceptance criteria - [ ] Complete healthcheck coverage inventory - [ ] List of modules missing healthchecks - [ ] Follow-up issues created for each gap (or one batch issue if many gaps) ## Do NOT read - Read module.yaml files only for health section - Do not deep-read full manifests ## Agent notes - Recommended executor: Gemini 3.5 Flash (pure investigation) - Size: Trivial (0 LOC — investigation + report) - Review tier: tier/trivial - Audit ref: state/audit/deepseek-2026-06-08-multiperspective.md §DR1
Collaborator

Iskra judgment

Field Value
Target pdurlej/platform#issue#765
Priority p2
Action codex_candidate
Scores reach 4 / impact 4 / confidence 5
Piotr fit high
Effort small
Labels judge/p2, judge/codex-candidate
Judge iskra via openclaw

Rationale: This is a strong P2 Codex candidate because a scoped healthcheck inventory can expose runtime blind spots and create targeted follow-ups without changing production behavior.

Caveat: Keep this as an inventory and follow-up creation task; do not add speculative healthchecks or alter module runtime definitions in the audit pass.

Structured openclaw.judge.v0 payload
<!-- openclaw.judge.v0 -->
{
  "confidence": 5,
  "effort_hint": "small",
  "escalation": {
    "kind": "none",
    "reason": ""
  },
  "evidence_refs": [
    {
      "note": "Issue scopes a ready runtime healthcheck inventory across module manifests.",
      "type": "forgejo",
      "value": "issue-title-body-labels-and-target-snapshot"
    },
    {
      "note": "Body defines concrete classifications for modules with healthchecks, without healthchecks, and not applicable orchestration cases.",
      "type": "forgejo",
      "value": "issue-body-scope-and-classification"
    },
    {
      "note": "Labels mark this as a small ready runtime chore with runtime risk.",
      "type": "snapshot",
      "value": "target-labels-and-target-snapshot"
    }
  ],
  "impact": 4,
  "judge_actor": {
    "name": "iskra",
    "runtime": "openclaw"
  },
  "judged_at": "2026-06-19T00:00:00Z",
  "labels_to_apply": [
    "judge/p2",
    "judge/codex-candidate"
  ],
  "piotr_fit": "high",
  "priority": "p2",
  "rationale_summary": "This is a strong P2 Codex candidate because a scoped healthcheck inventory can expose runtime blind spots and create targeted follow-ups without changing production behavior.",
  "reach": 4,
  "recommended_next_action": "codex_candidate",
  "rerun_reason": "no_prior_judgment",
  "schema": "openclaw.judge.v0",
  "target": {
    "kind": "issue",
    "number": 765,
    "repo": "pdurlej/platform"
  },
  "target_snapshot": {
    "body_hash": "sha256:74b988abba15c85347bd8144c5d1e2eafffdfa8dd61f3f2500175a75af193cf9",
    "commit_count": null,
    "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855",
    "head_sha": null,
    "labels": [
      "domain:runtime",
      "flow/ready",
      "kind/chore",
      "phase/03",
      "priority:p2",
      "risk/runtime",
      "size/small"
    ],
    "labels_hash": "sha256:908e4cd49c0008439ce4f98e33f036d0dd2ddb964f3f507837738f61b8d459db",
    "state": "open",
    "title_hash": "sha256:feb7f79e477bf636237b1752edb03ede44e8e86b78a82460967387693b18d28f",
    "updated_at": "2026-06-18T00:06:36+02:00"
  },
  "top_caveat": "Keep this as an inventory and follow-up creation task; do not add speculative healthchecks or alter module runtime definitions in the audit pass."
}
<!-- /openclaw.judge.v0 -->
### Iskra judgment | Field | Value | | --- | --- | | Target | `pdurlej/platform#issue#765` | | Priority | p2 | | Action | codex_candidate | | Scores | reach 4 / impact 4 / confidence 5 | | Piotr fit | high | | Effort | small | | Labels | `judge/p2`, `judge/codex-candidate` | | Judge | `iskra` via `openclaw` | **Rationale:** This is a strong P2 Codex candidate because a scoped healthcheck inventory can expose runtime blind spots and create targeted follow-ups without changing production behavior. **Caveat:** Keep this as an inventory and follow-up creation task; do not add speculative healthchecks or alter module runtime definitions in the audit pass. <details> <summary>Structured openclaw.judge.v0 payload</summary> ```json <!-- openclaw.judge.v0 --> { "confidence": 5, "effort_hint": "small", "escalation": { "kind": "none", "reason": "" }, "evidence_refs": [ { "note": "Issue scopes a ready runtime healthcheck inventory across module manifests.", "type": "forgejo", "value": "issue-title-body-labels-and-target-snapshot" }, { "note": "Body defines concrete classifications for modules with healthchecks, without healthchecks, and not applicable orchestration cases.", "type": "forgejo", "value": "issue-body-scope-and-classification" }, { "note": "Labels mark this as a small ready runtime chore with runtime risk.", "type": "snapshot", "value": "target-labels-and-target-snapshot" } ], "impact": 4, "judge_actor": { "name": "iskra", "runtime": "openclaw" }, "judged_at": "2026-06-19T00:00:00Z", "labels_to_apply": [ "judge/p2", "judge/codex-candidate" ], "piotr_fit": "high", "priority": "p2", "rationale_summary": "This is a strong P2 Codex candidate because a scoped healthcheck inventory can expose runtime blind spots and create targeted follow-ups without changing production behavior.", "reach": 4, "recommended_next_action": "codex_candidate", "rerun_reason": "no_prior_judgment", "schema": "openclaw.judge.v0", "target": { "kind": "issue", "number": 765, "repo": "pdurlej/platform" }, "target_snapshot": { "body_hash": "sha256:74b988abba15c85347bd8144c5d1e2eafffdfa8dd61f3f2500175a75af193cf9", "commit_count": null, "evidence_hash": "sha256:e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855", "head_sha": null, "labels": [ "domain:runtime", "flow/ready", "kind/chore", "phase/03", "priority:p2", "risk/runtime", "size/small" ], "labels_hash": "sha256:908e4cd49c0008439ce4f98e33f036d0dd2ddb964f3f507837738f61b8d459db", "state": "open", "title_hash": "sha256:feb7f79e477bf636237b1752edb03ede44e8e86b78a82460967387693b18d28f", "updated_at": "2026-06-18T00:06:36+02:00" }, "top_caveat": "Keep this as an inventory and follow-up creation task; do not add speculative healthchecks or alter module runtime definitions in the audit pass." } <!-- /openclaw.judge.v0 --> ``` </details>
Sign in to join this conversation.
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#765
No description provided.