[pkg-nagios-changes] [Git][nagios-team/icinga2][master] Disable flaky perfdata tests.

Bas Couwenberg (@sebastic) gitlab at salsa.debian.org
Thu Apr 23 05:35:31 BST 2026



Bas Couwenberg pushed to branch master at Debian Nagios Maintainer Group / icinga2


Commits:
573c8a7c by Bas Couwenberg at 2026-04-23T06:23:10+02:00
Disable flaky perfdata tests.

https://github.com/Icinga/icinga2/issues/10811

- - - - -


3 changed files:

- debian/changelog
- + debian/patches/perfdata-tests.patch
- debian/patches/series


Changes:

=====================================
debian/changelog
=====================================
@@ -1,3 +1,10 @@
+icinga2 (2.16.0-3) UNRELEASED; urgency=medium
+
+  * Disable flaky perfdata tests.
+    https://github.com/Icinga/icinga2/issues/10811
+
+ -- Bas Couwenberg <sebastic at debian.org>  Thu, 23 Apr 2026 06:01:34 +0200
+
 icinga2 (2.16.0-1) unstable; urgency=medium
 
   * New upstream release.


=====================================
debian/patches/perfdata-tests.patch
=====================================
@@ -0,0 +1,178 @@
+Description: Disable flaky perfdata tests.
+Author: Bas Couwenberg <sebastic at debian.org>
+Bug: https://github.com/Icinga/icinga2/issues/10811
+
+--- a/test/perfdata-elasticsearchwriter.cpp
++++ b/test/perfdata-elasticsearchwriter.cpp
+@@ -14,7 +14,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_elasti
+ 	*boost::unit_test::label("network")
+ )
+ 
+-BOOST_AUTO_TEST_CASE(connect)
++BOOST_AUTO_TEST_CASE(connect, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+@@ -40,7 +40,7 @@ BOOST_AUTO_TEST_CASE(connect)
+ 	PauseWriter();
+ }
+ 
+-BOOST_AUTO_TEST_CASE(pause_with_pending_work)
++BOOST_AUTO_TEST_CASE(pause_with_pending_work, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+--- a/test/perfdata-gelfwriter.cpp
++++ b/test/perfdata-gelfwriter.cpp
+@@ -14,7 +14,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_gelfwr
+ 	*boost::unit_test::label("network")
+ )
+ 
+-BOOST_AUTO_TEST_CASE(connect)
++BOOST_AUTO_TEST_CASE(connect, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+@@ -31,7 +31,7 @@ BOOST_AUTO_TEST_CASE(connect)
+ 	PauseWriter();
+ }
+ 
+-BOOST_AUTO_TEST_CASE(pause_with_pending_work)
++BOOST_AUTO_TEST_CASE(pause_with_pending_work, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+--- a/test/perfdata-graphitewriter.cpp
++++ b/test/perfdata-graphitewriter.cpp
+@@ -15,7 +15,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_graphi
+ 	*boost::unit_test::label("network")
+ )
+ 
+-BOOST_AUTO_TEST_CASE(connect)
++BOOST_AUTO_TEST_CASE(connect, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+@@ -30,7 +30,7 @@ BOOST_AUTO_TEST_CASE(connect)
+ 	PauseWriter();
+ }
+ 
+-BOOST_AUTO_TEST_CASE(pause_with_pending_work)
++BOOST_AUTO_TEST_CASE(pause_with_pending_work, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+--- a/test/perfdata-influxdbwriter.cpp
++++ b/test/perfdata-influxdbwriter.cpp
+@@ -13,7 +13,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_influx
+ 	*boost::unit_test::label("network")
+ )
+ 
+-BOOST_AUTO_TEST_CASE(connect)
++BOOST_AUTO_TEST_CASE(connect, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+@@ -33,7 +33,7 @@ BOOST_AUTO_TEST_CASE(connect)
+ 	PauseWriter();
+ }
+ 
+-BOOST_AUTO_TEST_CASE(pause_with_pending_work)
++BOOST_AUTO_TEST_CASE(pause_with_pending_work, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+--- a/test/perfdata-opentsdbwriter.cpp
++++ b/test/perfdata-opentsdbwriter.cpp
+@@ -15,7 +15,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_opents
+ 	*boost::unit_test::label("network")
+ )
+ 
+-BOOST_AUTO_TEST_CASE(connect)
++BOOST_AUTO_TEST_CASE(connect, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+@@ -36,7 +36,7 @@ BOOST_AUTO_TEST_CASE(connect)
+ 	PauseWriter();
+ }
+ 
+-BOOST_AUTO_TEST_CASE(pause_with_pending_work)
++BOOST_AUTO_TEST_CASE(pause_with_pending_work, * boost::unit_test::disabled())
+ {
+ 	ResumeWriter();
+ 
+--- a/test/perfdata-perfdatawriterconnection.cpp
++++ b/test/perfdata-perfdatawriterconnection.cpp
+@@ -51,7 +51,7 @@ BOOST_FIXTURE_TEST_SUITE(perfdata_connec
+ 
+ /* If there is no acceptor listening on the other side, connecting should fail.
+  */
+-BOOST_AUTO_TEST_CASE(connection_refused)
++BOOST_AUTO_TEST_CASE(connection_refused, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> p;
+ 	TestThread timeoutThread{[&]() {
+@@ -69,7 +69,7 @@ BOOST_AUTO_TEST_CASE(connection_refused)
+ /* The PerfdataWriterConnection connects automatically when sending the first data.
+  * In case of http we also need to support disconnecting and reconnecting.
+  */
+-BOOST_AUTO_TEST_CASE(ensure_connected)
++BOOST_AUTO_TEST_CASE(ensure_connected, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> disconnectedPromise;
+ 
+@@ -92,7 +92,7 @@ BOOST_AUTO_TEST_CASE(ensure_connected)
+ /* Verify that data can still be sent while CancelAfterTimeout is waiting and the timeout
+  * can be aborted when all data has been sent successfully.
+  */
+-BOOST_AUTO_TEST_CASE(finish_during_timeout)
++BOOST_AUTO_TEST_CASE(finish_during_timeout, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> p;
+ 
+@@ -128,7 +128,7 @@ BOOST_AUTO_TEST_CASE(finish_during_timeo
+  * in the kernel. But in that case the TLS handshake will be stuck, so we need to verify that a
+  * handshake can be interrupted by CancelAfterTimeout().
+  */
+-BOOST_AUTO_TEST_CASE(stuck_in_handshake)
++BOOST_AUTO_TEST_CASE(stuck_in_handshake, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> p;
+ 	TestThread timeoutThread{[&]() {
+@@ -151,7 +151,7 @@ BOOST_AUTO_TEST_CASE(stuck_in_handshake)
+  * No orderly TLS shutdown can be performed in this case, because the stream has been truncated.
+  * The server will need to handle this one on their own.
+  */
+-BOOST_AUTO_TEST_CASE(stuck_sending)
++BOOST_AUTO_TEST_CASE(stuck_sending, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> shutdownPromise;
+ 	std::promise<void> dataReadPromise;
+@@ -194,7 +194,7 @@ BOOST_AUTO_TEST_CASE(stuck_sending)
+  * in reality a server might not even do that. That case should be handled by our
+  * AsioTlsStream::GracefulDisconnect() function with an additional 10s timeout.
+  */
+-BOOST_AUTO_TEST_CASE(stuck_reading_response)
++BOOST_AUTO_TEST_CASE(stuck_reading_response, * boost::unit_test::disabled())
+ {
+ 	std::promise<void> shutdownPromise;
+ 	std::promise<void> requestReadPromise;
+@@ -234,7 +234,7 @@ BOOST_AUTO_TEST_CASE(stuck_reading_respo
+  * PerfdataWriterConnection should detect the disconnect, catch the exception and attempt to
+  * reconnect without exiting Send().
+  */
+-BOOST_AUTO_TEST_CASE(reconnect_failed)
++BOOST_AUTO_TEST_CASE(reconnect_failed, * boost::unit_test::disabled())
+ {
+ 	TestThread mockTargetThread{[&]() {
+ 		Accept();
+@@ -269,7 +269,7 @@ BOOST_AUTO_TEST_CASE(reconnect_failed)
+  * since on older boost versions the async_write() functions also accept non-const references, it
+  * doesn't hurt to ensure this with a test-case.
+  */
+-BOOST_AUTO_TEST_CASE(http_send_retry)
++BOOST_AUTO_TEST_CASE(http_send_retry, * boost::unit_test::disabled())
+ {
+ 	TestThread mockTargetThread{[&] {
+ 		Accept();


=====================================
debian/patches/series
=====================================
@@ -1,2 +1,3 @@
 21_config_changes
 postgres-checkcommand.patch
+perfdata-tests.patch



View it on GitLab: https://salsa.debian.org/nagios-team/icinga2/-/commit/573c8a7cd6c208fbb69d3ff49ecbc84646948387

-- 
View it on GitLab: https://salsa.debian.org/nagios-team/icinga2/-/commit/573c8a7cd6c208fbb69d3ff49ecbc84646948387
You're receiving this email because of your account on salsa.debian.org. Manage all notifications: https://salsa.debian.org/-/profile/notifications | Help: https://salsa.debian.org/help


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://alioth-lists.debian.net/pipermail/pkg-nagios-changes/attachments/20260423/b7bc196b/attachment-0001.htm>


More information about the pkg-nagios-changes mailing list