gstreamer1.0: skip a test that is known to be flaky

The aggregator testcase test_infinite_seek_50_src_live is known upstream
to be flaky[1] and when this fails in their CI they just ignore it.

It's failing often on our autobuilder, so disable the test case for now
until upstream have resolved this issue.

[ YOCTO #15054 ]

[1] https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/410

Signed-off-by: Ross Burton <ross.burton@arm.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
(cherry picked from commit c2c9cbc107e5428122ad26b5c478602f0c8c0fbe)
Signed-off-by: Steve Sakoman <steve@sakoman.com>

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0/skip-aggregator-test.patch b/meta/recipes-multimedia/gstreamer/gstreamer1.0/skip-aggregator-test.patch
new file mode 100644 (file)
index 0000000..8133751
--- /dev/null
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0/skip-aggregator-test.patch
@@ -0,0 +1,35 @@
+From 9b72aa7cdbc2a81cffc6f855933afe90c81046d5 Mon Sep 17 00:00:00 2001
+From: Ross Burton <ross.burton@arm.com>
+Date: Wed, 28 Feb 2024 12:40:34 +0000
+Subject: [PATCH] Skip aggregator test
+
+This test case is known to be flaky upstream[1] and often fails on the
+autobuilder[2], so skip it until this has been resolved upstream.
+
+[1] https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/410
+[2] https://bugzilla.yoctoproject.org/show_bug.cgi?id=15054
+
+Upstream-Status: Inappropriate
+Signed-off-by: Ross Burton <ross.burton@arm.com>
+---
+ tests/check/libs/aggregator.c | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/tests/check/libs/aggregator.c b/tests/check/libs/aggregator.c
+index 1f2c5b4..27b3ac7 100644
+--- a/tests/check/libs/aggregator.c
++++ b/tests/check/libs/aggregator.c
+@@ -1475,7 +1475,9 @@ gst_aggregator_suite (void)
+   tcase_add_test (general, test_flushing_seek);
+   tcase_add_test (general, test_infinite_seek);
+   tcase_add_test (general, test_infinite_seek_50_src);
+-  tcase_add_test (general, test_infinite_seek_50_src_live);
++  // This test case is known to be flaky, remove it until resolved:
++  // https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/410
++  // tcase_add_test (general, test_infinite_seek_50_src_live);
+   tcase_add_test (general, test_linear_pipeline);
+   tcase_add_test (general, test_two_src_pipeline);
+   tcase_add_test (general, test_timeout_pipeline);
+-- 
+2.34.1
+

diff --git a/meta/recipes-multimedia/gstreamer/gstreamer1.0_1.22.9.bb b/meta/recipes-multimedia/gstreamer/gstreamer1.0_1.22.9.bb
index b4ab6ad10ced53d811ff82a5fadc83f0478cafb7..9d634e35dc3be6a781866fc1dcfbfb07c9e4b51d 100644 (file)
--- a/meta/recipes-multimedia/gstreamer/gstreamer1.0_1.22.9.bb
+++ b/meta/recipes-multimedia/gstreamer/gstreamer1.0_1.22.9.bb
@@ -21,6 +21,7 @@ SRC_URI = "https://gstreamer.freedesktop.org/src/gstreamer/gstreamer-${PV}.tar.x
            file://0002-tests-add-support-for-install-the-tests.patch \
            file://0003-tests-use-a-dictionaries-for-environment.patch;striplevel=3 \
            file://0004-tests-add-helper-script-to-run-the-installed_tests.patch;striplevel=3 \
+           file://skip-aggregator-test.patch \
            "
 SRC_URI[sha256sum] = "1e7124d347e8cdc80f08ec1d370c201be513002af1102bb20e83c5279cb48ebd"