public inbox for git-commits@fedoraproject.org
help / color / mirror / Atom feed
From: Florian Weimer <fweimer@redhat.com>
To: git-commits@fedoraproject.org
Subject: [rpms/glibc] rawhide: Fail the build if there are unexpected test failures (#2449608)
Date: Tue, 01 Sep 2026 20:00:13 GMT	[thread overview]
Message-ID: <178829281328.1.14542808289477977593.rpms-glibc-2f41fe788ac6@fedoraproject.org> (raw)

            A new commit has been pushed.

            Repo   : rpms/glibc
            Branch : rawhide
            Commit : 2f41fe788ac62e89a5d94e8ad6101620e0d86804
            Author : Florian Weimer <fweimer@redhat.com>
            Date   : 2026-09-01T13:45:29+02:00
            Stats  : +45/-11 in 8 file(s)
            URL    : https://src.fedoraproject.org/rpms/glibc/c/2f41fe788ac62e89a5d94e8ad6101620e0d86804?branch=rawhide

            Log:
            Fail the build if there are unexpected test failures (#2449608)

Put existing i686 and ppc64le failures on the allow-list.

---
diff --git a/allowed-failures-aarch64.txt b/allowed-failures-aarch64.txt
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/allowed-failures-aarch64.txt

diff --git a/allowed-failures-i386.txt b/allowed-failures-i386.txt
new file mode 100644
index 0000000..f55bf4f
--- /dev/null
+++ b/allowed-failures-i386.txt
@@ -0,0 +1,5 @@
+math/test-double-compoundn # swbz#33239
+math/test-float-compoundn # swbz#33239
+math/test-float32-compoundn # swbz#33239
+math/test-float32x-compoundn # swbz#33239
+math/test-float64-compoundn # swbz#33239

diff --git a/allowed-failures-noarch.txt b/allowed-failures-noarch.txt
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/allowed-failures-noarch.txt

diff --git a/allowed-failures-ppc64le.txt b/allowed-failures-ppc64le.txt
new file mode 100644
index 0000000..a671a5b
--- /dev/null
+++ b/allowed-failures-ppc64le.txt
@@ -0,0 +1,6 @@
+math/test-double-acospi # GCC PR118541
+math/test-float128-acospi # GCC PR118541
+math/test-float32x-acospi # GCC PR118541
+math/test-float64-acospi # GCC PR118541
+math/test-float64x-acospi # GCC PR118541
+math/test-ldouble-acospi # GCC PR118541

diff --git a/allowed-failures-riscv64.txt b/allowed-failures-riscv64.txt
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/allowed-failures-riscv64.txt

diff --git a/allowed-failures-s390x.txt b/allowed-failures-s390x.txt
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/allowed-failures-s390x.txt

diff --git a/allowed-failures-x86_64.txt b/allowed-failures-x86_64.txt
new file mode 100644
index 0000000..e69de29
--- /dev/null
+++ b/allowed-failures-x86_64.txt

diff --git a/glibc.spec b/glibc.spec
index 41cecc6..326c809 100644
--- a/glibc.spec
+++ b/glibc.spec
@@ -152,7 +152,7 @@ Version: %{glibcversion}
 # - It allows using the Release number without the %%dist tag in the dependency
 #   generator to make the generated requires interchangeable between Rawhide
 #   and ELN (.elnYY < .fcXX).
-%global baserelease 1
+%global baserelease 2
 Release: %{baserelease}%{?dist}
 
 # Licenses:
@@ -226,6 +226,16 @@ Source11: parse-SUPPORTED.py
 # Include in the source RPM for reference.
 Source12: ChangeLog.old
 
+# Flaky tests that do not fail the build.  The noarch file is used on all
+# architectures.
+Source20: allowed-failures-noarch.txt
+Source21: allowed-failures-aarch64.txt
+Source22: allowed-failures-i386.txt
+Source23: allowed-failures-ppc64le.txt
+Source24: allowed-failures-riscv64.txt
+Source25: allowed-failures-s390x.txt
+Source26: allowed-failures-x86_64.txt
+
 # glibc_ldso: ABI-specific program interpreter name.  Used for debuginfo
 # extraction (wrap-find-debuginfo.sh) and smoke testing ($run_ldso below).
 #
@@ -1989,19 +1999,20 @@ split_sysroot_file_list \
 # synchronization issues with make and shell tracing output if
 # standard output and standard error are different pipes.
 run_tests () {
-  # This hides a test suite build failure, which should be fatal.  We
-  # check "Summary of test results:" below to verify that all tests
-  # were built and run.
-  %make_build check |& tee rpmbuild.check.log >&2
-  test -n tests.sum
-  if ! grep -Eq '^\s+=== Summary of results ===$' rpmbuild.check.log ; then
-    echo "FAIL: test suite build of target: $(basename "$(pwd)")" >& 2
-    exit 1
-  fi
+  # Delay exiting from a test suite failure after reporting.
+  local ret=0
+  %make_build check || ret=$?
   set +x
   grep -v ^PASS: tests.sum > rpmbuild.tests.sum.not-passing || true
+  # Only used FAILED TESTS if there were failures not on the allow-list.
+  local label
+  if test $ret -eq 0; then
+      label=NON-PASSING
+  else
+      label=FAILED
+  fi
   if test -n rpmbuild.tests.sum.not-passing ; then
-    echo ===================FAILED TESTS===================== >&2
+    echo ===================$label TESTS===================== >&2
     echo "Target: $(basename "$(pwd)")" >& 2
     cat rpmbuild.tests.sum.not-passing >&2
     while read failed_code failed_test ; do
@@ -2020,6 +2031,8 @@ run_tests () {
   echo "* System call consistency checks:" >&2
   cat misc/tst-syscall-list.out >&2
   set -x
+
+  test $ret -eq 0 || exit $ret
 }
 
 # Increase timeouts
@@ -2027,6 +2040,13 @@ export TIMEOUTFACTOR=16
 parent=$$
 echo ====================TESTING=========================
 
+# Install the allow-list into the source tree.  The
+# architecture-specific file is optional (but all files should exist
+# on the regular Fedora architectures).
+cp %{_sourcedir}/allowed-failures-noarch.txt allowed-failures.txt
+(cat %{_sourcedir}/allowed-failures-%{_arch}.txt || true) \
+    >> allowed-failures.txt
+
 # Default libraries.
 pushd build-%{target}
 run_tests
@@ -2459,6 +2479,9 @@ update_gconv_modules_cache ()
 %endif
 
 %changelog
+* Tue Sep 01 2026 Florian Weimer  <fweimer@redhat.com> - 2.44.9000-2
+- Fail the build if there are unexpected test failures (#2449608)
+
 * Tue Aug 25 2026 Frédéric Bérat <fberat@redhat.com> - 2.44.9000-1
 - Auto-sync with upstream branch master,
   commit 53e5f02a6bca102884daed79abd1540aa9b9d19c:

                 reply	other threads:[~2026-09-01 20:00 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=178829281328.1.14542808289477977593.rpms-glibc-2f41fe788ac6@fedoraproject.org \
    --to=fweimer@redhat.com \
    --cc=git-commits@fedoraproject.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox