-
Notifications
You must be signed in to change notification settings - Fork 175
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
[8.0] added ulimit -n 1048575 to pilotwrapper content #7891
Conversation
bd13ad8
to
49df0ba
Compare
How comfortable are you about the test coverage of this? |
There are tests: https://github.com/DIRACGrid/DIRAC/actions/runs/11911498836 |
There was some lengthy discussion, the details of which I don't remember, because I got sidetracked, about ulimits especially at RAL-LCG2 in the GridPP ops meeting today. LHCb was mentioned. Maybe you could send some hand hacked jobs to RAL-LCG2 for testing ? |
https://arc-ce02.gridpp.rl.ac.uk:443/arex/VMOLDmA7eX6nc1XDjq68JLkqYugZkqABFKDmF2FKDmTEIKDmny5KHm I will check them later on |
Sweep summary Sweep ran in https://github.com/DIRACGrid/DIRAC/actions/runs/11929890350 Successful:
|
closes #7657
BEGINRELEASENOTES
*WMS
CHANGE: added ulimit -n 1048575 to pilotwrapper content
ENDRELEASENOTES