Skip to content

halide-cache-server: monitoring - #9

Merged
RMPR merged 1 commit into
masterfrom
halide-server-monitoring
Oct 6, 2026
Merged

RMPR merged 1 commit into
masterfrom
halide-server-monitoring

Conversation

@RMPR

@RMPR RMPR commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

This is mostly vibed. I verified visually that I get what I want, but not much beyond that. Attached is a screenshot of what it looks like

image

@RMPR
RMPR added this pull request to stack #11 October 5, 2026 09:55
@RMPR
RMPR requested a review from vawale October 5, 2026 09:59
@RMPR
RMPR force-pushed the halide-server-monitoring branch 2 times, most recently from 8fc800b to 194f090 Compare October 5, 2026 12:57
@RMPR
RMPR force-pushed the halide-server-monitoring branch from 194f090 to 87effa2 Compare October 6, 2026 07:26
Base automatically changed from halide-server to master October 6, 2026 07:54
DISCLAIMER: Completely vibe-coded feel free to ignore. I didn't look at
the code myself and I don't expect anyone to. I judge this one by output

- /v1/history: per-minute counters and store size for 24 hours
- /v1/clients: per-address counters, with the hostname the client now
  sends in X-Halide-Cache-Client
- eviction ages in /v1/stats: a histogram of how long evicted entries had
  gone unused, the signal that --size is too small
- /metrics in Prometheus format
- a self-contained dashboard at / showing all of the above

Totals, per-minute buckets and per-client rows share one Counters type
behind the existing lock.
@RMPR
RMPR force-pushed the halide-server-monitoring branch from 87effa2 to 732eb95 Compare October 6, 2026 07:54
@RMPR
RMPR merged commit fb6e6a1 into master Oct 6, 2026
18 checks passed
@RMPR
RMPR deleted the halide-server-monitoring branch October 6, 2026 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant