Fix: tests: app unregistering is not guaranteed by app lifetime
authorJonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Mon, 23 Aug 2021 21:12:28 +0000 (17:12 -0400)
committerJérémie Galarneau <jeremie.galarneau@efficios.com>
Fri, 17 Dec 2021 05:31:09 +0000 (00:31 -0500)
commit65d1f746e7dbd27cbc5208734ceecab7c5fb4a3d
tree1a96368f6c35685cd75066659b873fbbbf84c935
parentd1892e1a33645be2d173b8d92808664fb4f6bbce
Fix: tests: app unregistering is not guaranteed by app lifetime

Observed issue
==============

The per-pid timer based rotation tests fail on a minimal ptest
yocto image.

The test suite report that the second archive is not empty as it
expects.

Note that the yocto/OE image is running under QEMU without
KVM.

Cause
=====

Since the image is running under QEMU without KVM, the overall
processing capability of the VM is quite limited.

The test seems to assume that between the first and the second rotation
the app will be unregistered by the time the second rotation is issued.

Note that the observable lifetime of an app is not equal to the
lttng-sessiond/consumerd app visibility since we deal with app
unregistration via a polling mechanism.

Note, that as far as I understand, this is a testing issue only.

It is still relevant in the context of rotation to validate that the second
rotation archive does NOT contain info for a "dead" app under per-pid
configuration.

Solution
========

Move the rotation timer operation after the app is registered and
considered unregistered from the point of view of
lttng-sessiond/lttng-consumerd. This should give us a more robust
approach.

Known drawbacks
=========

None.

References
==========

Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien@efficios.com>
Signed-off-by: Jérémie Galarneau <jeremie.galarneau@efficios.com>
Change-Id: Ie8c542d29ef8bdb325efc05de14e80b179c68754
tests/regression/tools/mi/test_mi
tests/regression/tools/relayd-grouping/test_ust
tests/regression/tools/rotation/test_save_load_mi
tests/regression/tools/rotation/test_ust
tests/regression/tools/save-load/test_autoload
tests/utils/utils.sh
This page took 0.02913 seconds and 5 git commands to generate.