Skip to content

Commit

Permalink
ZTS: alloc_class.ksh must wait for the process to exit
Browse files Browse the repository at this point in the history
The alloc_class_* tests may fail on Linux with an EBUSY error if
`zfs destroy` is run before the `dd` process has had a chance to
terminate.  Wait on the pid after the `kill -9` to make sure.

When testing I didn't observe any failures for the alloc_class
tests.  Remove them from the exceptions list, the CI can verify
they are passing on all platforms.

Signed-off-by: Brian Behlendorf <[email protected]>
Issue openzfs#11888
  • Loading branch information
behlendorf committed Dec 16, 2021
1 parent 8fdc6f6 commit 929af43
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 5 deletions.
5 changes: 0 additions & 5 deletions tests/test-runner/bin/zts-report.py.in
Original file line number Diff line number Diff line change
Expand Up @@ -289,11 +289,6 @@ if sys.platform.startswith('freebsd'):
})
elif sys.platform.startswith('linux'):
maybe.update({
'alloc_class/alloc_class_009_pos': ['FAIL', known_reason],
'alloc_class/alloc_class_010_pos': ['FAIL', known_reason],
'alloc_class/alloc_class_011_neg': ['FAIL', known_reason],
'alloc_class/alloc_class_012_pos': ['FAIL', known_reason],
'alloc_class/alloc_class_013_pos': ['FAIL', '11888'],
'cli_root/zfs_rename/zfs_rename_002_pos': ['FAIL', known_reason],
'cli_root/zpool_expand/zpool_expand_001_pos': ['FAIL', known_reason],
'cli_root/zpool_expand/zpool_expand_005_pos': ['FAIL', known_reason],
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,7 @@ function display_status
((ret |= $?))

kill -9 $pid
wait $pid 2> /dev/null

return $ret
}

0 comments on commit 929af43

Please sign in to comment.