Re: Adding CI to our tree

From: Andres Freund <andres(at)anarazel(dot)de>
To: 0010203112132233 <boekewurm(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>
Subject: Re: Adding CI to our tree
Date: 2021-10-02 00:10:53
Message-ID: 20211002001053.cw2p2shqmyzygsms@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2021-10-02 01:49:39 +0200, 0010203112132233 wrote:
> On Sat, 2 Oct 2021 at 00:28, Andres Freund <andres(at)anarazel(dot)de> wrote:
> > New contributors (and quite a bit of older ones too) IMO expect to be able to
> > see whether their changes work as-is, without sending a patch to the list.
>
> Have they checked 'make installcheck-world'? I believe that is one of
> the first action items on the 'So, you want to become a developer?'
> wiki page, after downloading the sources. Of course, that is limited
> to only the environment of the user, but that's what they're generally
> developing for, right?

If you want to get a change into postgres, it almost always needs to actually
work on all operating systems, and always needs to at least not cause build
failures on all platforms.

> Furthermore, after looking it through, I think that Cirrus is an
> unfortunate choice as a CI platform of preference, as you cannot use
> it without access to Github (which is problematic for people located
> in certain localities due to USA regulations).

I agree that it's not optimal that cirrus isn't available on all git hosting
platforms. Hence saying that I think it's likely we'd end up adding a few more
platforms over time. If we factor the meat of the work into an helper script,
so that the CI specific bit is just a couple invocation of that script, it's
not a lot of overhead to have 2-3 CI platforms.

> If we're going to include CI configuration for private use, I'd prefer if it
> were a CI that can be enjoyed in private without pushing code to a 3rd
> party.

FWIW, you can use cirrus locally on your machine:
https://github.com/cirruslabs/cirrus-cli

It'll not be able to run all kinds of tasks though (e.g. no windows docker on
a linux host, dealing with the license costs for that presumably would be
nontrivial).

> Lastly, I consider CI configuration similar to IDE configuration: each
> developer has their own preferred tools which they use, but we don't
> favour one over the other. We don't include IDE-specific configuration
> files either, or at least, the policy is against that.
>
> So, I greatly appreciate the effort, but I don't think this is
> something that should be committed into core. Maybe as a dedicated
> wiki page detailing configurations for CI, similar to the Buildfarm
> page?

That doesn't scale - I've actually added CI to all my substantial development
work in my private branches, and it's pretty annoying to need to do so every
time. And there's many developers who won't go through the effort most of the
time.

It's not like this forces you to use cirrus or anything. For people that don't
want to use CI, It'll make cfbot a bit more effective (because people can
adjust what it tests as appropriate for $patch), but that's it.

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tatsuo Ishii 2021-10-02 02:33:47 Problem with CF app?
Previous Message Kevin Burke 2021-10-02 00:09:17 Better context for "TAP tests not enabled" error message