Skip to content

Conversation

mikelittle
Copy link
Contributor

@mikelittle mikelittle commented May 8, 2025

Work in progress

Implement a default memory limit for containers if one is not specified.
For now, it applies the same limit across all services.
But as enhanced-search now specifies its own docker-compose config, that is not affected by this. The ES container has a memory limit, but the Kibana container does not.
Similarly, the proxy container is independent and does not specify a limit.

Implements #818

 Work in progress

 Implement a default memory limit for containers if one is not specified.
 For now it applies the same limit across all services.
 But as enhanced-search now specifies its own docker-compose config, that is not affected by this. The es contaainer has a memry limit but the kibana container does not.
 Similarly, the proxy container is independent and does not specify a limit.

 Implemements #818
Copy link

@svandragt svandragt left a comment

Choose a reason for hiding this comment

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

Linttert issues not withstanding, looks good.

@svandragt
Copy link

svandragt commented Jul 30, 2025

This is affecting several people on a weekly basis, would be great if it can be merged and backported. When the local machine is running out of memory all kinds of freezes and worse happen due to memory exhaustion and the oom killer.

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