Skip to content

feat(observability): Add container memory utilization - #223

Merged
nfebe merged 2 commits into
mainfrom
feat/container-memory-utilization
Aug 25, 2026
Merged

feat(observability): Add container memory utilization#223
nfebe merged 2 commits into
mainfrom
feat/container-memory-utilization

Conversation

@nfebe

@nfebe nfebe commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Add a container memory utilization metric so percentage alert thresholds use each container's effective memory limit.

@sourceant

sourceant Bot commented Aug 25, 2026

Copy link
Copy Markdown

Code Review Summary

This PR introduces a new container.memory.utilization metric (calculated as usage/limit * 100). This allows users to set percentage-based alerts that are relative to each container's specific memory constraints. The implementation is consistent across the entire observability stack, including storage, alerting, OTLP, and Prometheus exporters.

🚀 Key Improvements

  • Introduced MetricMemoryUtilization calculation in the storage layer with safety checks for zero limits.
  • Unified formatting for percentage-based metrics in alert messages.
  • Full integration with OTLP and Prometheus exporters including proper units (%) and help documentation.
  • Comprehensive test coverage across API, storage, and alert formatting layers.

@sourceant sourceant Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review complete. No specific code suggestions were generated. See the overview comment for a summary.

@nfebe
nfebe merged commit 57ac8bd into main Aug 25, 2026
6 checks passed
@nfebe
nfebe deleted the feat/container-memory-utilization branch August 25, 2026 17:25
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