Skip to content

Fix default memory limit monitor

Nikola Milojevic requested to merge fix-puma-worker-rss-memory-limit-monitor into master

What does this MR do and why?

In !102300 (merged) we introduced RssMemoryLimit monitor. It compares Gitlab::Metrics::System.memory_usage_rss[:total] which is measured in bytes against provided memory_limit.

The issue is that we are passing memory_limit in megabytes.

This MR fixes and converts memory limit configured with PUMA_WORKER_MAX_MEMORY env variable to bytes

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports