mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Ahmad Fatoum <a.fatoum@barebox.org>
To: barebox@lists.infradead.org
Cc: Ahmad Fatoum <a.fatoum@barebox.org>
Subject: [PATCH] ci: pytest: print labgrid console output inline
Date: Wed,  6 Aug 2025 14:46:20 +0200	[thread overview]
Message-ID: <20250806124620.2800123-1-a.fatoum@barebox.org> (raw)

Makes it easier to debug issues instead of having to download the
console_main files:

- Print the console_main files inline on error
- Collect all console_main files at the end

Signed-off-by: Ahmad Fatoum <a.fatoum@barebox.org>
---
 .github/workflows/test-labgrid-pytest.yml | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/.github/workflows/test-labgrid-pytest.yml b/.github/workflows/test-labgrid-pytest.yml
index 38bd7126bd86..adec3defc1d7 100644
--- a/.github/workflows/test-labgrid-pytest.yml
+++ b/.github/workflows/test-labgrid-pytest.yml
@@ -96,6 +96,7 @@ jobs:
         done
 
     - name: labgrid-pytest
+      if: steps.build.outcome == 'success'
       run: |
         export KBUILD_OUTPUT=build-${{matrix.arch}}-${{matrix.defconfig}}
 
@@ -113,17 +114,19 @@ jobs:
           --junitxml=$cfg.tests.xml --lg-log=log/$cfg $extraargs
         done
 
+    - name: Print Labgrid Console Output
+      if: always()
+      run: |
+        for i in ${{matrix.lgenv}}; do
+          grep -wq '\(QEMUDriver\|ExternalConsoleDriver\):' "$i" || continue
+
+          cfg=$(basename $i .yaml)
+          cat log/$cfg/* || true
+        done
+
     - name: Publish Test Results
       uses: EnricoMi/publish-unit-test-result-action@v2
       if: steps.build.outcome == 'success'
       with:
         check_name: "Test Results (${{matrix.defconfig}})"
         files: ./*.tests.xml
-
-    - name: Publish Labgrid Log Results
-      uses: actions/upload-artifact@v4
-      if: steps.build.outcome == 'success'
-      with:
-        name: console-log-${{matrix.defconfig}}
-        path: log/
-        if-no-files-found: error
-- 
2.39.5




             reply	other threads:[~2025-08-06 12:47 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-08-06 12:46 Ahmad Fatoum [this message]
2025-08-07  6:02 ` Sascha Hauer

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=20250806124620.2800123-1-a.fatoum@barebox.org \
    --to=a.fatoum@barebox.org \
    --cc=barebox@lists.infradead.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