Skip to content

permission error fix in test_cluster_spooler.py teardown #1648

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.

Sign up for GitHub

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
David-Baddeley merged 3 commits into from
Mar 6, 2026
Merged

permission error fix in test_cluster_spooler.py teardown #1648

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
e1e97d0
let the dataserver release the fill if we permission error
barentine Mar 6, 2026
aa5a3a8
Merge branch 'master' of https://github.com/python-microscopy/python-…
barentine Mar 6, 2026
ff36d3b
Merge branch 'master' of github.com:python-microscopy/python-microsco…
David-Baddeley Mar 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions tests/PYME/IO/test_cluster_spooler.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,8 +38,11 @@ def teardown_module():
proc.kill()

print('Killed all servers')

shutil.rmtree(tmp_root)
def _wait_for_poll_exit(func, path, _):
# wait for the server process to exit and release the file lock before we try to delete the spooler dir
import time
time.sleep(1)
shutil.rmtree(tmp_root, onerror=_wait_for_poll_exit)



Expand Down
Loading