Skip to content

selftests: Set a timeout for lsof #13621

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

Merged
merged 1 commit into from
Jul 28, 2025

Conversation

The-Compiler
Copy link
Member

In case of hanging filesystems (e.g. unreachable sshfs), lsof can hang for a long time (3 minutes in my case). Set a more reasonable timeout and skip the affected test (like already done with other problems regarding lsof) in such a case.

In case of hanging filesystems (e.g. unreachable sshfs), `lsof` can
hang for a long time (3 minutes in my case). Set a more reasonable
timeout and skip the affected test (like already done with other problems
regarding lsof) in such a case.
@psf-chronographer psf-chronographer bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Jul 27, 2025
Copy link
Member

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lovely, thanks :)

@nicoddemus nicoddemus added the backport 8.4.x apply to PRs at any point; backports the changes to the 8.4.x branch label Jul 28, 2025
@nicoddemus nicoddemus merged commit 836cfba into pytest-dev:main Jul 28, 2025
35 of 36 checks passed
Copy link

patchback bot commented Jul 28, 2025

Backport to 8.4.x: 💚 backport PR created

✅ Backport PR branch: patchback/backports/8.4.x/836cfbac1f49c069191b5859fbc1c1763e5077ea/pr-13621

Backported as #13623

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

patchback bot pushed a commit that referenced this pull request Jul 28, 2025
In case of hanging filesystems (e.g. unreachable sshfs), `lsof` can
hang for a long time (3 minutes in my case). Set a more reasonable
timeout and skip the affected test (like already done with other problems
regarding lsof) in such a case.

(cherry picked from commit 836cfba)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 8.4.x apply to PRs at any point; backports the changes to the 8.4.x branch bot:chronographer:provided (automation) changelog entry is part of PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants