Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plugin-procfs automatically increases file descriptors soft limit #118

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ahuret
Copy link

@ahuret ahuret commented Feb 24, 2025

Automatically increase alumet process's file descriptors soft limit when plugin-procfs starts in order to prevent 'Too many open files' error that happened because procfs plugin opens lot of files.

It starts without condition (we could, later, add a configuration option - plugin side - to disable this behavior in some scenarios where administrator would like to manage this limit precisely).

The current implementation will check for the process hard limit and will update soft limit to reach the hard limit. This way the user has still control on the overall process limit - by defining the hard limit.

A future implementation could also be located in Alumet's core to perform the same operation for all the plugins.

@ahuret ahuret force-pushed the auto-increase-file-descriptors-soft-limit branch from fb711e3 to 28ed047 Compare February 24, 2025 13:51
@ahuret
Copy link
Author

ahuret commented Feb 24, 2025

Closes #65

@TheElectronWill TheElectronWill added the A:existing-plugin area: an existing plugin label Feb 25, 2025
@TheElectronWill TheElectronWill self-requested a review March 3, 2025 13:22
Copy link
Contributor

@TheElectronWill TheElectronWill left a comment

Choose a reason for hiding this comment

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

The commit message should be:

feat(plugin-procfs): automatically increases file descriptors soft limit

@ahuret ahuret force-pushed the auto-increase-file-descriptors-soft-limit branch from 28ed047 to b480e39 Compare March 3, 2025 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A:existing-plugin area: an existing plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants