Skip to content

Fix log of dead processes in reap_stray_processes#68928

Open
m-czernek wants to merge 1 commit intosaltstack:masterfrom
m-czernek:fixture-fix
Open

Fix log of dead processes in reap_stray_processes#68928
m-czernek wants to merge 1 commit intosaltstack:masterfrom
m-czernek:fixture-fix

Conversation

@m-czernek
Copy link
Copy Markdown
Contributor

If reap_stray_processes cannot kill a process,
it logs which process it gived up on. However,
if the process exits after the alive check and
before the log, this causes psutil.NoSuchProcess
exception

Commits signed with GPG?

Yes/No

If reap_stray_processes cannot kill a process,
it logs which process it gived up on. However,
if the process exits after the alive check and
before the log, this causes psutil.NoSuchProcess
exception
@m-czernek m-czernek requested a review from a team as a code owner April 13, 2026 09:44
@dwoz dwoz added the test:full Run the full test suite label Apr 13, 2026
m-czernek added a commit to openSUSE/salt that referenced this pull request Apr 14, 2026
* Fix test failures

salt/states/pkgrepo.py:
- saltstack/salt@919994f

tests/pytests/functional/modules/test_pkg.py:
- saltstack/salt@7835f3a
- saltstack/salt@6dc6cc3

tests/integration/modules/test_gem.py:
- Started failing because after several years, brass released a new
  version
- saltstack/salt@1d4f9dc

tests/pytests/unit/modules/test_aptpkg.py:
- Was broken after our latest code upgrade

* Fix reap_stray_processes fixture error
- saltstack/salt#68928
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

test:coverage test:full Run the full test suite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants