| 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 |
| 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 |