Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
slapos slapos
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 122
    • Merge requests 122
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • slaposslapos
  • Merge requests
  • !1465

You need to sign in or sign up before continuing.
Merged
Created Oct 31, 2023 by Levin Zimmermann@levin.zimmermannMaintainer

WIP: software/erp5: Add rlimit_nofile (?) option (2)

  • Overview 12
  • Commits 3
  • Changes 10

Hello,

due to the reasons stated in !1451 (comment 193296) merge request !1451 (merged) was dropped by force pushing to master branch.

So I open this up again. We need to find a consensuses regarding the name of the option which sets the soft limit of open file descriptors to its hard limit. Once we find this consensuses I can push a fixup commit to ERP5, then adjust the name here, adjust the commit messages according to !1451 (comment 193296) and finally hopefully merge again.

In !1451 (comment 193297) @jm suggested rlimit-nofile++ or

something more Nexediish

I was thinking of a verbose --max-soft-fds-limit. Are there any other concrete ideas or opinions?

Best, Levin

++ I'd go for - instead of _, because the other zopewsgi options use - and @jerome suggested using - as a convention in SlapOS parameters instead of _. So we could even use the same style for both ERP5 and SlapOS, which makes it easier to memorize.

/cc @vpelletier

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: addnofilelimitsetter2
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7