Reviewed by: Matt Ahrens <m...@delphix.com> Reviewed by: Chris Williamson <chris.william...@delphix.com>
Test 13 would fail because of attempts to zpool destroy -f a pool that was still busy. Changed those calls to destroy_pool which does a retry loop, and the problem is no longer reproducible. Also removed some non functional code in the test which is why it was originally commented out by placing it after the call to log_pass. Test 14 would fail because sometimes the check for a degraded pool would complete before the pool had changed state. Changed the logic to check in a loop with a timeout and the problem is no longer reproducible. Upstream bug: DLPX-32291 You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/583 -- Commit Summary -- * 9245 zfs-test failures: slog_013_pos and slog_014_pos -- File Changes -- M usr/src/test/zfs-tests/include/libtest.shlib (20) M usr/src/test/zfs-tests/tests/functional/slog/slog.kshlib (8) M usr/src/test/zfs-tests/tests/functional/slog/slog_013_pos.ksh (17) M usr/src/test/zfs-tests/tests/functional/slog/slog_014_pos.ksh (15) -- Patch Links -- https://github.com/openzfs/openzfs/pull/583.patch https://github.com/openzfs/openzfs/pull/583.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/583 ------------------------------------------ openzfs: openzfs-developer Permalink: https://openzfs.topicbox.com/groups/developer/discussions/T5104e463417473b6-Mc0f73d824a1d78a4af9177e1 Delivery options: https://openzfs.topicbox.com/groups