Andrew Geissler | 5199d83 | 2021-09-24 16:47:35 -0500 | [diff] [blame] | 1 | From 8d9daede0882d239b0a47b0f7a6db68ba4934a7d Mon Sep 17 00:00:00 2001 |
| 2 | From: Alexander Kanavin <alex@linutronix.de> |
| 3 | Date: Sat, 4 Sep 2021 13:57:39 +0200 |
| 4 | Subject: [PATCH] tests: wait some more before analysing traces or starting |
| 5 | tracing |
| 6 | |
| 7 | Otherwise, there are sporadic race failures where lttng tracing |
| 8 | is stopped before all expected events are collected or is started too soon, e.g.: |
| 9 | |
| 10 | PASS: tools/tracker/test_event_tracker 205 - Traced application stopped. |
| 11 | PASS: tools/tracker/test_event_tracker 206 - Stop lttng tracing for session |
| 12 | PASS: tools/tracker/test_event_tracker 207 - Destroy session tracker |
| 13 | FAIL: tools/tracker/test_event_tracker 208 - Validate empty trace |
| 14 | |
| 15 | PASS: ust/namespaces/test_ns_contexts_change 42 - Stop lttng tracing for session mnt_ns |
| 16 | PASS: ust/namespaces/test_ns_contexts_change 43 - Destroy session mnt_ns |
| 17 | PASS: ust/namespaces/test_ns_contexts_change 44 - Wait after kill session daemon |
| 18 | PASS: ust/namespaces/test_ns_contexts_change 45 - Validate trace for event mnt_ns = 4026531840, 1000 events |
| 19 | PASS: ust/namespaces/test_ns_contexts_change 46 - Read a total of 1000 events, expected 1000 |
| 20 | PASS: ust/namespaces/test_ns_contexts_change 47 - Validate trace for event mnt_ns = 4026532303, 233 events |
| 21 | FAIL: ust/namespaces/test_ns_contexts_change 48 - Read a total of 233 events, expected 1000 |
| 22 | |
| 23 | This is a hack; issue should be fixed upstream with explicit syncs. |
| 24 | It has been reported here: https://bugs.lttng.org/issues/1217 |
| 25 | |
| 26 | Upstream-Status: Inappropriate [needs a real fix] |
| 27 | Signed-off-by: Alexander Kanavin <alex@linutronix.de> |
| 28 | --- |
| 29 | tests/regression/tools/tracker/test_event_tracker | 8 ++++++++ |
| 30 | tests/regression/ust/namespaces/test_ns_contexts_change | 2 ++ |
| 31 | 2 files changed, 10 insertions(+) |
| 32 | |
| 33 | diff --git a/tests/regression/tools/tracker/test_event_tracker b/tests/regression/tools/tracker/test_event_tracker |
| 34 | index feb3787..a0f2257 100755 |
| 35 | --- a/tests/regression/tools/tracker/test_event_tracker |
| 36 | +++ b/tests/regression/tools/tracker/test_event_tracker |
| 37 | @@ -130,6 +130,8 @@ function test_event_vpid_tracker() |
| 38 | |
| 39 | prepare_"$domain"_app |
| 40 | |
| 41 | +sleep 5 |
| 42 | + |
| 43 | start_lttng_tracing_ok |
| 44 | |
| 45 | if [ "$expect_event" -eq 1 ]; then |
| 46 | @@ -173,6 +175,8 @@ function test_event_pid_tracker() |
| 47 | |
| 48 | prepare_"$domain"_app |
| 49 | |
| 50 | +sleep 5 |
| 51 | + |
| 52 | start_lttng_tracing_ok |
| 53 | |
| 54 | if [ "$expect_event" -eq 1 ]; then |
| 55 | @@ -275,6 +279,8 @@ function test_event_vpid_track_untrack() |
| 56 | |
| 57 | prepare_"$domain"_app |
| 58 | |
| 59 | +sleep 5 |
| 60 | + |
| 61 | start_lttng_tracing_ok |
| 62 | |
| 63 | lttng_track_"$domain"_ok "--vpid ${CHILD_PID}" |
| 64 | @@ -315,6 +321,8 @@ function test_event_pid_track_untrack() |
| 65 | |
| 66 | prepare_"$domain"_app |
| 67 | |
| 68 | +sleep 5 |
| 69 | + |
| 70 | start_lttng_tracing_ok |
| 71 | |
| 72 | lttng_track_"$domain"_ok "--pid ${CHILD_PID}" |
| 73 | diff --git a/tests/regression/ust/namespaces/test_ns_contexts_change b/tests/regression/ust/namespaces/test_ns_contexts_change |
| 74 | index c0af15e..b111bfe 100755 |
| 75 | --- a/tests/regression/ust/namespaces/test_ns_contexts_change |
| 76 | +++ b/tests/regression/ust/namespaces/test_ns_contexts_change |
| 77 | @@ -79,6 +79,8 @@ function test_ns() |
| 78 | |
| 79 | touch "$file_sync_before_last" |
| 80 | |
| 81 | +sleep 5 |
| 82 | + |
| 83 | # stop and destroy |
| 84 | stop_lttng_tracing_ok "$session_name" |
| 85 | destroy_lttng_session_ok "$session_name" |
| 86 | -- |
| 87 | 2.20.1 |
| 88 | |