aboutsummaryrefslogtreecommitdiffstats
path: root/tests/zfs-tests
diff options
context:
space:
mode:
authorBrian Behlendorf <[email protected]>2020-11-03 09:19:45 -0800
committerGitHub <[email protected]>2020-11-03 09:19:45 -0800
commit123d2803bfb9b754cee7e721a67a151380efd7e7 (patch)
treee82830760f207cea82f9a29752d6d824b7da7c3d /tests/zfs-tests
parent59b687232798a4da0e7647cca19a308aa82b5fad (diff)
ZTS: Wait on all events in events_001_pos.ksh
The events_001_pos.ksh test case can fail because it's possible, and correct, for the config_sync event to be posted after the last "expected" event. To accommodate this the run_and_verify() function has been updated to wait for all non-history events, not just the last event. This does not increase the run time of the test as long as all the events do get generated. Reviewed-by: George Melikov <[email protected]> Signed-off-by: Brian Behlendorf <[email protected]> Closes #11147
Diffstat (limited to 'tests/zfs-tests')
-rw-r--r--tests/zfs-tests/tests/functional/events/events_common.kshlib12
1 files changed, 10 insertions, 2 deletions
diff --git a/tests/zfs-tests/tests/functional/events/events_common.kshlib b/tests/zfs-tests/tests/functional/events/events_common.kshlib
index 5f4c629dc..9c5879183 100644
--- a/tests/zfs-tests/tests/functional/events/events_common.kshlib
+++ b/tests/zfs-tests/tests/functional/events/events_common.kshlib
@@ -119,8 +119,16 @@ function run_and_verify
# If the only event is history then we don't observe zed debug log
if [[ "${events[0]}" != "sysevent.fs.zfs.history_event" ]]; then
- # wait for the last event to show up in the debug log
- log_must file_wait_event $ZED_DEBUG_LOG ${events[-1]}
+ # wait for all the non-history events to show up in the
+ # debug log, all-debug.sh filters history events.
+ for event in ${events[*]}; do
+ if [[ "$event" == \
+ "sysevent.fs.zfs.history_event" ]]; then
+ continue
+ fi
+
+ log_must file_wait_event $ZED_DEBUG_LOG "$event"
+ done
log_must cp $ZED_DEBUG_LOG $TMP_EVENTS_ZED
log_must test -s $TMP_EVENTS_ZED