Buildroot Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] support/misc/gitlab-ci.yml.in: print error log if the runtime test fail
@ 2023-11-11 11:37 Romain Naour
  2023-12-03 19:21 ` Peter Korsgaard
  0 siblings, 1 reply; 2+ messages in thread
From: Romain Naour @ 2023-11-11 11:37 UTC (permalink / raw)
  To: buildroot; +Cc: Romain Naour

When an error occurs, the gitlab-ci job log doesn't contain any useful
information than the name of the failing test:

  FAIL: test_run (tests.package.test_python_paho_mqtt.TestPythonPahoMQTT)

In order to encourage contributors to investigate issues reported by
gitlab-ci, we want to print the last lines of the log file (build or
runtime).

Unfortunately, gitlab-ci job log completely strips lines ending with
CRCRLF [1][2]. We have to take a look at the gitlab-ci raw log to see
the complete log [3].

To workaround this issue, remove crlf from qemu serial stdio log
while printing in the gitlab-ci job log (we don't want to change
the log file generated by support/testing/run-tests and saved as
artefacts).

[1] https://gitlab.com/gitlab-org/gitlab/-/issues/218771
[2] https://gitlab.com/kubu93/buildroot/-/jobs/5492937691
[3] https://gitlab.com/kubu93/buildroot/-/jobs/5492937691/raw

Cc: Peter Korsgaard <peter@korsgaard.com>
Signed-off-by: Romain Naour <romain.naour@smile.fr>
---
 support/misc/gitlab-ci.yml.in | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/support/misc/gitlab-ci.yml.in b/support/misc/gitlab-ci.yml.in
index 4d9acbc3d3..db206432aa 100644
--- a/support/misc/gitlab-ci.yml.in
+++ b/support/misc/gitlab-ci.yml.in
@@ -103,7 +103,16 @@ before_script:
     script:
         - TEST_CASE_NAME=${CI_JOB_NAME}
         - echo "Starting runtime test ${TEST_CASE_NAME}"
-        - ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${TEST_CASE_NAME}
+        - |
+            ./support/testing/run-tests -o test-output/ -d test-dl/ -k --timeout-multiplier 10 ${TEST_CASE_NAME} || {
+                echo 'Failed runtime test last output'
+                if [ -f test-output/*-run.log ]; then
+                    tail -200 test-output/*-run.log | sed 's/\r\r$//'
+                else
+                    tail -200 test-output/*-build.log
+                fi
+                exit 1
+            }
     retry:
         max: 2
         when:
-- 
2.41.0

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Buildroot] [PATCH] support/misc/gitlab-ci.yml.in: print error log if the runtime test fail
  2023-11-11 11:37 [Buildroot] [PATCH] support/misc/gitlab-ci.yml.in: print error log if the runtime test fail Romain Naour
@ 2023-12-03 19:21 ` Peter Korsgaard
  0 siblings, 0 replies; 2+ messages in thread
From: Peter Korsgaard @ 2023-12-03 19:21 UTC (permalink / raw)
  To: Romain Naour; +Cc: buildroot

>>>>> "Romain" == Romain Naour <romain.naour@smile.fr> writes:

 > When an error occurs, the gitlab-ci job log doesn't contain any useful
 > information than the name of the failing test:

 >   FAIL: test_run (tests.package.test_python_paho_mqtt.TestPythonPahoMQTT)

 > In order to encourage contributors to investigate issues reported by
 > gitlab-ci, we want to print the last lines of the log file (build or
 > runtime).

 > Unfortunately, gitlab-ci job log completely strips lines ending with
 > CRCRLF [1][2]. We have to take a look at the gitlab-ci raw log to see
 > the complete log [3].

 > To workaround this issue, remove crlf from qemu serial stdio log
 > while printing in the gitlab-ci job log (we don't want to change
 > the log file generated by support/testing/run-tests and saved as
 > artefacts).

 > [1] https://gitlab.com/gitlab-org/gitlab/-/issues/218771
 > [2] https://gitlab.com/kubu93/buildroot/-/jobs/5492937691
 > [3] https://gitlab.com/kubu93/buildroot/-/jobs/5492937691/raw

 > Cc: Peter Korsgaard <peter@korsgaard.com>
 > Signed-off-by: Romain Naour <romain.naour@smile.fr>

Committed to next, thanks.

-- 
Bye, Peter Korsgaard
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2023-12-03 19:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-11 11:37 [Buildroot] [PATCH] support/misc/gitlab-ci.yml.in: print error log if the runtime test fail Romain Naour
2023-12-03 19:21 ` Peter Korsgaard

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox