389-ds-base

#50116 Issue 48064 - Fix various issues in disk monitoring test suite
Closed by spichugi. Opened by vashirov.
vashirov/389-ds-base fix_disk_monitoring_hang  into  master

Download 50116.patch

Bug Description:
1. Some tests were hanging after disk monitoring test suite was executed,
because we didn't do a proper cleanup: log directories were still
mounted after test has ended.
2. Deprecation warnings from re module.

Fix Description:
1. Change scope of the fixture to 'module' instead of 'function'.
Split setup fixture to have a separate fixture for log reset for each
test.
2. Use raw strings for re functions' parameters.

https://pagure.io/389-ds-base/issue/48064

Reviewed by: ???

LGTM, @spichugi thoughts?

Ack. And the tests pass.

rebased onto 3e42a5ec0b87c83576916403042e21a78bebd4aa

rebased onto a3d35b9ddd3989fe09e3a98d6f13d8f6f28df6d4

Pull-Request has been merged by vashirov

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3175

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.