From a6ea32a79893b6ee694d036f3bc29a02a79d51e8 Mon Sep 17 00:00:00 2001
From: Richard van der Hoff <1389908+richvdh@users.noreply.github.com>
Date: Mon, 2 Aug 2021 21:06:34 +0100
Subject: [PATCH] Fix the `tests-done` github actions step, again (#10512)

---
 .github/workflows/tests.yml | 21 ++++++++++++---------
 changelog.d/10512.misc      |  1 +
 2 files changed, 13 insertions(+), 9 deletions(-)
 create mode 100644 changelog.d/10512.misc

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 0a62c62d0..239553ae1 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -367,13 +367,16 @@ jobs:
       - name: Set build result
         env:
           NEEDS_CONTEXT: ${{ toJSON(needs) }}
-        # the `jq` incantation dumps out a series of "<job> <result>" lines
+        # the `jq` incantation dumps out a series of "<job> <result>" lines.
+        # we set it to an intermediate variable to avoid a pipe, which makes it
+        # hard to set $rc.
         run: |
-          set -o pipefail
-          jq -r 'to_entries[] | [.key,.value.result] | join(" ")' \
-                          <<< $NEEDS_CONTEXT |
-              while read job result; do
-                  if [ "$result" != "success" ]; then
-                      echo "::set-failed ::Job $job returned $result"
-                  fi
-              done
+          rc=0
+          results=$(jq -r 'to_entries[] | [.key,.value.result] | join(" ")' <<< $NEEDS_CONTEXT)
+          while read job result ; do
+              if [ "$result" != "success" ]; then
+                  echo "::set-failed ::Job $job returned $result"
+                  rc=1
+              fi
+          done <<< $results
+          exit $rc
diff --git a/changelog.d/10512.misc b/changelog.d/10512.misc
new file mode 100644
index 000000000..c012e89f4
--- /dev/null
+++ b/changelog.d/10512.misc
@@ -0,0 +1 @@
+Update the `tests-done` Github Actions status.