Re: Building with musl in CI and the build farm

From: Wolfgang Walther <walther(at)technowledgy(dot)de>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Peter Eisentraut <peter(at)eisentraut(dot)org>, Andres Freund <andres(at)anarazel(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Building with musl in CI and the build farm
Date: 2024-04-04 19:09:30
Message-ID: 9b7cc292-9feb-46c9-af2c-ecd55fbd413a@technowledgy.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Tom Lane:
>> You'd have to commit a failing patch first to break CI for all other
>> developers.
>
> No, what I'm more worried about is some change in the environment
> causing the build to start failing. When that happens, it'd better
> be an environment that many of us are familiar with and can test/fix.

The way I understand how this work is, that the images for the VMs in
which those CI tasks run, are not just dynamically updated - but are
actually tested before they are used in CI. So the environment doesn't
just change suddenly.

See e.g. [1] for a pull request to the repo containing those images to
update the linux debian image from bullseye to bookworm. This is exactly
the image we're talking about. Before this image is used in postgres CI,
it's tested and confirmed that it actually works there. If one of the
jobs was using musl - that would be tested as well. So this job would
not just suddenly start failing for everybody.

I do see the "familiarity" argument for the SanityCheck task, but for a
different reason: Even though it's unlikely for this job to fail for
musl specific reasons - if you're not familiar with musl and can't
easily test it locally, you might not be able to tell immediately
whether it's musl specific or not. If musl was run in one of the later
jobs, it's much different: You see all tests failing - alright, not musl
specific. You see only the musl test failing - yeah, musl problem. This
should give developers much more confidence looking at the results.

Best,

Wolfgang

[1]: https://github.com/anarazel/pg-vm-images/pull/91

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Thibaut BOULDOIRE 2024-04-05 09:35:45 Sequence name with capital letters issue
Previous Message Tom Lane 2024-04-04 15:19:11 Re: Building with musl in CI and the build farm

Browse pgsql-hackers by date

  From Date Subject
Next Message Tomas Vondra 2024-04-04 19:11:27 Re: incremental backup breakage in BlockRefTableEntryGetBlocks
Previous Message Jeff Davis 2024-04-04 18:55:51 Re: Improve eviction algorithm in ReorderBuffer