Repair AI usage exporter poller health and cleanup #12
Loading…
x
Reference in New Issue
Block a user
No description provided.
Delete Branch "wt/t_1e95ea6d"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
/healthzreflect poller thread liveness and bounded progress, independently of upstream quota successRoot cause
The second bounded
wait()after killing a stuck Codex subprocess could raiseTimeoutExpiredoutside the collector's narrow exception handler. That terminated the daemon poller while the unconditional health endpoint kept the sidecar Ready and stale metrics available indefinitely. Metrics disconnects were also not handled at the response boundary.Validation
kustomize buildand client dry-run passed for bothservices/hermesandservices/monitoringgit diff --checkpassedThe broader local quality profile was also inspected: docs, Ruff smell, and managed-file coverage passed. Its hygiene check remains failed on four unchanged files already over the 500-line limit; two unit tests initially observed mounted live health/credential readiness and passed when rerun with isolated nonexistent paths.
Security and rollout
Failure logs contain only fixed provider names and exception class names, never exception messages, provider responses, OAuth contents, credentials, or account identity. No paid API key or interactive
/statusscraping was added. This PR is unmerged and no live workload was changed; the pod-template config revision will trigger the exporter restart only after reviewed Flux reconciliation.hermes-automation referenced this pull request2026-08-17 10:14:46 +00:00
Repair validation note: PR #14 must merge first, then PR #12 should be refreshed against that main. Exact synthetic composition of PR #14 head
8c6e3acdacfollowed by PR #12 headed43dbaa7fmerged cleanly. The composed quality gate passed 1,056 tests with zero files over 500 LOC; exporter modules had 100%, 99%, and 100% line/branch coverage. Hermes and monitoring renders plus client and kustomize-controller server dry-runs passed. Until PR #14 lands, PR #12 alone retains only the four known oversized baseline-test hygiene failures that PR #14 splits. No deploy, reconcile, build, publish, or merge was performed.