Re: CI and test improvements

From: Justin Pryzby <pryzby(at)telsasoft(dot)com>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: vignesh C <vignesh21(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Andres Freund <andres(at)anarazel(dot)de>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Noah Misch <noah(at)leadboat(dot)com>, Anastasia Lubennikova <lubennikovaav(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Melanie Plageman <melanieplageman(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Daniel Gustafsson <daniel(at)yesql(dot)se>, samay sharma <smilingsamay(at)gmail(dot)com>
Subject: Re: CI and test improvements
Date: 2024-02-13 19:10:20
Message-ID: Zcu-nD7ybe7P0Hjg@pryzbyj2023
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Wed, Jan 31, 2024 at 11:59:21AM +0100, Alvaro Herrera wrote:
> On 2024-Jan-31, vignesh C wrote:
> > Are we planning to do anything more on this? I was not sure if we
> > should move this to next commitfest or return it.
> Well, the patches don't apply anymore since .cirrus.tasks.yml has been
> created. However, I'm sure we still want [some of] the improvements
> to the tests in [1]. I can volunteer to rebase the patches in time for the
> March commitfest, if Justin is not available to do so. If you can
> please move it forward to the March cf and set it WoA, I'd appreciate
> that.

The patches are rebased. A couple were merged since I last rebased them
~10 months ago. The freebsd patch will probably be obsoleted by a patch
of Thomas.

On Mon, Mar 13, 2023 at 07:39:52AM +0100, Peter Eisentraut wrote:
> On 03.02.23 15:26, Justin Pryzby wrote:
> > 9baf41674ad pg_upgrade: tap test: exercise --link and --clone
> This seems like a good idea.

On Wed, Mar 15, 2023 at 10:58:41AM +0100, Peter Eisentraut wrote:
> > [PATCH 4/8] pg_upgrade: tap test: exercise --link and --clone
> I haven't been able to get any changes to the test run times outside
> of noise from this. But some more coverage is sensible in any case.
> I'm concerned that with this change, the only platform that tests
> --copy is Windows, but Windows has a separate code path for copy. So
> we should leave one Unix platform to test --copy. Maybe have FreeBSD
> test --link and macOS test --clone and leave the others with --copy?

I addressed Peter's comments, but haven't heard further.

The patch to show HTML docs artifacts may be waiting for Andres' patch
to convert CompilerWarnings to meson.

It may also be waiting on cfbot to avoid squishing all the patches

I sent various patches to cfbot but haven't heard back.


Attachment Content-Type Size
0001-cirrus-windows-add-compiler_warnings_script.patch text/x-diff 1.8 KB
0002-cirrus-002_pg_upgrade-exercise-link-and-clone.patch text/x-diff 1.1 KB
0003-WIP-ci-meson-allow-showing-only-failed-tests.patch text/x-diff 4.5 KB
0004-WIP-cirrus-upload-changed-html-docs-as-artifacts.patch text/x-diff 2.8 KB
0005-html-index-file.patch text/x-diff 2.6 KB
0006-WIP-show-changed-docs-with-meson.patch text/x-diff 2.5 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2024-02-13 19:14:01 Re: Fix overflow hazard in interval rounding
Previous Message Tomas Vondra 2024-02-13 19:00:59 Re: index prefetching