Skip to content

apollo_dashboard: use container_memory_rss for Pod Memory Limit Utilization panel#14201

Open
yonatanw-starkware wants to merge 2 commits into
mainfrom
yonatanw/dashboard-memory-limit-panel-rss
Open

apollo_dashboard: use container_memory_rss for Pod Memory Limit Utilization panel#14201
yonatanw-starkware wants to merge 2 commits into
mainfrom
yonatanw/dashboard-memory-limit-panel-rss

Conversation

@yonatanw-starkware
Copy link
Copy Markdown

Summary

  • Fixes the Pod Memory Limit Utilization dashboard panel to use container_memory_rss instead of container_memory_working_set_bytes
  • working_set includes active file-backed page cache which the kernel can reclaim, causing the panel to read near 100% even when there is no real OOM risk
  • container_memory_rss (anon memory only) correctly reflects actual memory pressure and OOM risk
  • The Pod Memory Request Utilization panel is intentionally left unchanged

Test plan

  • Verify dev_grafana.json regenerated correctly (2-line change in Pod Memory Limit Utilization panel expr)
  • Confirm Pod Memory Limit Utilization panel shows realistic values in Grafana after deploy

🤖 Generated with Claude Code

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@yonatanw-starkware yonatanw-starkware changed the base branch from main to main-v0.14.2 May 26, 2026 11:50
@yonatanw-starkware yonatanw-starkware changed the base branch from main-v0.14.2 to main May 26, 2026 12:08
…zation panel

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yonatanw-starkware yonatanw-starkware force-pushed the yonatanw/dashboard-memory-limit-panel-rss branch from f0b5404 to 719cd57 Compare May 26, 2026 12:13
… alerts

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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.

2 participants