Re: Upload only the failed tests logs to the Postgres CI (Cirrus CI)

From: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
To: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Upload only the failed tests logs to the Postgres CI (Cirrus CI)
Date: 2026-06-05 12:23:52
Message-ID: CAN55FZ07AefTV_D2bCZae5jtQOQD1QByNe3FbXvM9Lq166c4og@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Thu, 9 Apr 2026 at 11:01, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> wrote:
> On Tue, 7 Apr 2026 at 19:27, Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> wrote:
> >
> Here is v2, which addresses the issues I mentioned earlier.
>
> I have updated the design since this will only be used for Meson
> builds. I decided to invoke the script using 'meson compile
> clear_testrun_folder', as this avoids the need to manually call
> python3. This approach prevents issues on NetBSD, where the python3
> environment may not be found.
>
> I don’t expect the 'ci_meson_clear_testrun_folder' script to be used
> outside of CI, so I placed it in the 'src/tools/ci/' directory.
> However, if you think it could be useful beyond CI, we could consider
> renaming it and moving it to 'src/tools/'.

Here is the v3, it is modified for Github Actions.

I implemented it as another step which will run if the job fails.
Alternatively, we can put 'the clearing command' into the
meson_test_world step, this makes the change simpler but then this
command will run although tests don't fail.

With this change, the compressed size of the log decreases from ~8MB
to ~1MB. Also, it is much easier to find which tests failed.

You can see example GHA run, which intentionally failed:
https://github.com/nbyavuz/postgres/actions/runs/27013199064

--
Regards,
Nazir Bilal Yavuz
Microsoft

Attachment Content-Type Size
v3-0001-ci-Don-t-collect-successful-tests-logs.patch text/x-patch 5.1 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Ilmar Yunusov 2026-06-05 12:37:26 Re: File locks for data directory lockfile in the context of Linux namespaces
Previous Message Ilmar Yunusov 2026-06-05 12:02:28 Re: [Bug Report + Patch] File descriptor leak when io_method=io_uring