Skip to content

Raise open file descriptors limit to avoid problem with pystack#40110

Merged
peterfpeterson merged 1 commit into
mantidproject:ornl-nextfrom
peterfpeterson:raise_open_file_descriptors_limit_ornlnext
Oct 14, 2025
Merged

Raise open file descriptors limit to avoid problem with pystack#40110
peterfpeterson merged 1 commit into
mantidproject:ornl-nextfrom
peterfpeterson:raise_open_file_descriptors_limit_ornlnext

Conversation

@peterfpeterson
Copy link
Copy Markdown
Member

This is a version of #40108 into ornl-next

…idproject#40108)

### Description of work

The recent problems with running `pystack` in the error reporter have
been traced back to the fact that `pystack` was trying to open more file
descriptors than the limit set on most linux systems. A pr into
`pystack` has been made to improve their error messages
(bloomberg/pystack#260).

When this is a problem, you get a lot of this kind of output
```
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../.././libMantidJson.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../.././libMantidJson.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../.././libMantidJson.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../.././libMantidJson.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidTypes.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidTypes.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidTypes.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidTypes.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidPythonInterfaceCore.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidPythonInterfaceCore.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidPythonInterfaceCore.so
ERROR(process_core): Cannot open ELF file /home/bya67386/mambaforge/envs/test_htlp_links/lib/python3.11/site-packages/mantid/kernel/../../../../libMantidPythonInterfaceCore.
```

The issue started after the first new instrument view pr went in (this
commit
mantidproject@8733e98#diff-4a6d6571d2c59705d8a673667672de2d7dcc3c756c61a3f0195acdefac52fe34).
We're not that sure why, but it could be the addition of new libraries
such as `pyvista`.

This pr raises the limit (equivalent to `ulimit -n <value>`) for the
outer mantid process, which may eventually launch the error reporter.

### To test:

Package to test is at `mamba install
jhaigh0/label/raise_open_files_limit::mantidworkbench`

It would be good to test on idaaas where the core dump location is
already set up.
Run Segfault and check that the pystack output has been captured in the
stacktrace field.
@peterfpeterson peterfpeterson added the ornl-next A copy of a branch into the ornl-next fork label Oct 14, 2025
@peterfpeterson peterfpeterson enabled auto-merge (squash) October 14, 2025 15:49
@peterfpeterson peterfpeterson merged commit 85d167d into mantidproject:ornl-next Oct 14, 2025
9 of 11 checks passed
@peterfpeterson peterfpeterson deleted the raise_open_file_descriptors_limit_ornlnext branch October 14, 2025 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ornl-next A copy of a branch into the ornl-next fork

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants