Skip to content
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

tests/test_inotify_buffer.py tests fail in 1-core-VM #823

Closed
bmwiedemann opened this issue Jul 29, 2021 · 2 comments
Closed

tests/test_inotify_buffer.py tests fail in 1-core-VM #823

bmwiedemann opened this issue Jul 29, 2021 · 2 comments

Comments

@bmwiedemann
Copy link

While working on reproducible builds for openSUSE, I found that our python-watchdog-2.1.3 package failed 1 test when running in a 1-core-VM (usually this happens due to differences in scheduling/timing)

To reproduce

maybe run tests as taskset 1 pytest

or on Debian or openSUSE run

osc checkout openSUSE:Factory/python-watchdog && cd $_
osc build --vm-type=kvm --noservice --clean -j1 standard

Actual behavior

4 test errors

 =========================== short test summary info ============================
 ERROR tests/test_inotify_buffer.py::test_move_to - AssertionError: assert not...
 ERROR tests/test_inotify_buffer.py::test_move_internal - AssertionError: asse...
 ERROR tests/test_inotify_buffer.py::test_move_internal_batch - AssertionError...
 ERROR tests/test_inotify_buffer.py::test_close_should_terminate_thread - Asse...
 =========== 100 passed, 6 skipped, 1 deselected, 4 errors in 29.42s ============

You can find the full build log with errors at https://rb.zq1.de/temp/python-watchdog-j1-buildlog.txt

@BoboTiG
Copy link
Collaborator

BoboTiG commented Jul 29, 2021

Attached the log file for history.
python-watchdog-j1-buildlog.txt


The issue is not about 1-core, but about OSError: [Errno 9] Bad file descriptor. It is not the first time someone reported such issue 🤔

event_buffer = os.read(self._inotify_fd, event_buffer_size)

@BoboTiG
Copy link
Collaborator

BoboTiG commented Jul 29, 2021

I close that issue as duplicate of #805. Thanks for the report @bmwiedemann :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants