fix: Only call get_all_running_processes_info when needed

[mihaitodor]

Do not call get_all_running_processes_info() unless the log level requires printing debug logs.

When running micromamba in a specific production environment, we keep getting Segmentation fault (core dumped) and, by adding more logs in the code, I was able to narrow it down to this particular log entry, where it tries to print the currently running processes. I can confirm that the crash doesn't occur in get_all_running_processes_info() and it is something in the MessageLogger implementation, but I have a hard time narrowing it down since I don't have access to run gdb in that environment or collect core dumps. I believe it is something to do with the length of the resulting string, but I've not been able to get it to reproduce reliably in a unit test. However, we don't use debug logs and, in this case, I think the code should skip this log clause completely, which should unblock us.

Description

Type of Change

• Bugfix
• Feature / enhancement
• CI / Documentation
• Maintenance

Checklist

• My code follows the general style and conventions of the codebase, ensuring consistency
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have run pre-commit run --all locally in the source folder and confirmed that there are no linter errors.
• I have added tests that prove my fix is effective or that my feature works
• New and existing tests pass locally with my changes

[codecov]

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 63.12%. Comparing base (bc2ff56) to head (78b0212).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
libmamba/src/core/run.cpp	0.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #4135   +/-   ##
=======================================
  Coverage   63.11%   63.12%           
=======================================
  Files         316      316           
  Lines       38806    38807    +1     
  Branches     2986     2987    +1     
=======================================
+ Hits        24494    24498    +4     
+ Misses      14241    14238    -3     
  Partials       71       71           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jjerphan]

Thank you, @mihaitodor.

Indeed get_all_running_processes_info only must be called when needed.

[mihaitodor]

@jjerphan Thank you for confirming! I fixed the linter error (should've checked again before pushing).

[jjerphan]

I have rebased your PR to have #4137 in.