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
fix: handle benmark bug that arise with singularity #1671
Conversation
running snakemake with singularity results in invalid benchmark statistics, since snakemake doesn't have read access to all pids.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
@johanneskoester I have updated the commit with more handling for events when no benchmark can be collected. When debbuing is turned on and if any process cannot be processed information about which process that can and cannot be processed will be printed. |
Description
Running snakemake with singularity results in (nearly) empty benchmark statistics. benchmark tries to read a process with name "starter-suid" which raise a AccessDenied event, resulting in that the other processes belonging to the snakemake process aren't read. Wrapping the proc.memory_full_info() in a try-statement that fires a continue event enables the other processes to be parsed.
This should help solve issue: #1267
QC
docs/
) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).