code.ossystems Code Review - openembedded-core.git/blob

   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