All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marat Khalili <marat.khalili@huawei.com>
To: Bruce Richardson <bruce.richardson@intel.com>,
	"dev@dpdk.org" <dev@dpdk.org>
Cc: Aaron Conole <aconole@redhat.com>
Subject: RE: [PATCH v2] ci: improve logging of unit tests runs
Date: Mon, 19 Jan 2026 13:20:59 +0000	[thread overview]
Message-ID: <c45f79ce16944a3daa0fa45aba830a1c@huawei.com> (raw)
In-Reply-To: <20260119115607.225929-1-bruce.richardson@intel.com>

> -----Original Message-----
> From: Bruce Richardson <bruce.richardson@intel.com>
> Sent: Monday 19 January 2026 11:56
> To: dev@dpdk.org
> Cc: Bruce Richardson <bruce.richardson@intel.com>; Aaron Conole <aconole@redhat.com>
> Subject: [PATCH v2] ci: improve logging of unit tests runs
> 
> To help track down issues reported by CI test runs, add some additional
> meson flags to the run of the fast-tests:
> 
> --no-stdsplit:
>   keep the stdout and stderr output interleaved, rather
>   than separated in the logs, to make following what is happening during
>   a test run easier.
> --print-errorlogs:
>   when an error with a unit test does occur, output to the terminal the
>   last N lines of output. This allows us to see inline what may have
>   failed without always needing to check log files.
> 
> Signed-off-by: Bruce Richardson <bruce.richardson@intel.com>
> ---
> V2: Drop max-lines flag, since it's only introduced in very recent meson
>     versions.
> ---
>  .ci/linux-build.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/.ci/linux-build.sh b/.ci/linux-build.sh
> index ceead259f3..084d9642fc 100755
> --- a/.ci/linux-build.sh
> +++ b/.ci/linux-build.sh
> @@ -218,7 +218,7 @@ fi
>  if [ "$RUN_TESTS" = "true" ]; then
>      failed=
>      configure_coredump
> -    sudo meson test -C build --suite fast-tests -t 3 || failed="true"
> +    sudo meson test -C build --suite fast-tests -t 3 --no-stdsplit --print-errorlogs || failed="true"
>      catch_coredump
>      catch_ubsan DPDK:fast-tests build/meson-logs/testlog.txt
>      check_traces
> --
> 2.51.0

Acked-by: Marat Khalili <marat.khalili@huawei.com>

(Using both flags in our local CI for long time.)

  reply	other threads:[~2026-01-19 13:21 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-01-19 10:41 [PATCH] ci: improve logging of unit tests runs Bruce Richardson
2026-01-19 11:56 ` [PATCH v2] " Bruce Richardson
2026-01-19 13:20   ` Marat Khalili [this message]
2026-01-20 13:19   ` Aaron Conole
2026-01-20 16:24     ` Patrick Robb
2026-01-21  9:14   ` David Marchand

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=c45f79ce16944a3daa0fa45aba830a1c@huawei.com \
    --to=marat.khalili@huawei.com \
    --cc=aconole@redhat.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.