pg.ddx.io pgsql-hackers@postgresql.org mailing list archive
help / color / mirror / Atom feedFrom: Justin Pryzby <pryzby@telsasoft.com>
To: Andres Freund <andres@anarazel.de>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Robert Haas <robertmhaas@gmail.com>
Cc: Andrew Dunstan <andrew@dunslane.net>
Cc: pgsql-hackers@postgresql.org, Thomas Munro <thomas.munro@gmail.com>
Cc: Melanie Plageman <melanieplageman@gmail.com>
Cc: Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Cc: Daniel Gustafsson <daniel@yesql.se>
Subject: Re: Adding CI to our tree
Date: Sat, 26 Feb 2022 21:10:57 -0600
Message-ID: <20220227031057.GD25269@telsasoft.com> (raw)
In-Reply-To: <20220227025000.mjmfimls5gnz7em7@alap3.anarazel.de>
References: <20220204050403.GL23027@telsasoft.com>
<20220206032339.tuyo534rfbvu4mbh@alap3.anarazel.de>
<20220212220640.GL31460@telsasoft.com>
<20220212222625.aph3ft466ntifrpi@alap3.anarazel.de>
<20220213214213.GS31460@telsasoft.com>
<20220213220709.vjz5rziuhfdpqxrg@alap3.anarazel.de>
<20220226025116.GJ9008@telsasoft.com>
<20220227010908.vz2a7dmfzgwg742w@alap3.anarazel.de>
<20220227024352.GC25269@telsasoft.com>
<20220227025000.mjmfimls5gnz7em7@alap3.anarazel.de>
On Sat, Feb 26, 2022 at 06:50:00PM -0800, Andres Freund wrote:
> Hi,
>
> On 2022-02-26 20:43:52 -0600, Justin Pryzby wrote:
> > This doesn't do the right thing - I just tried.
> > https://cirrus-ci.org/guide/writing-tasks/#environment-variables
> > | changesInclude function can be very useful for skipping some tasks when no changes to sources have been made since the last successful Cirrus CI build.
>
> > That means it will not normally rebuild docs (and then this still requires
> > resolving the "base branch").
>
> Why would we want to rebuild docs if they're the same as in the last build for
> the same branch? For cfbot purposes each commit is independent from the prior
> commit, so it should rebuild it every time if the CF entry has changes to the
> docs.
I did git commit --amend --no-edit and repushed to github to trigger a new CI
run, and it did this: https://github.com/justinpryzby/postgres/runs/5347878714
This is in a branch with changes to doc. I wasn't intending it to skip
building docs on this branch just because the same, changed docs were
previously built.
Why wouldn't the docs be built following the same logic as the rest of the
sources ? If someone renames or removes an xref target, shouldn't CI fail on
its next run for a patch which tries to reference it ? It would fail on the
buildfarm, and I think one major use for the CI is to minimize the post-push
cleanup cycles.
Are you sure about cfbot ? AIUI cirrus would see that docs didn't change
relative to the previous run for branch: commitfest/NN/MMMM.
--
Justin
view thread (142+ messages) latest in thread
Message-ID: <20220227031057.GD25269@telsasoft.com>
Permalink: ../20220227031057.GD25269@telsasoft.com/
Also on: postgresql.org/message-id/20220227031057.GD25269@telsasoft.com
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-hackers@postgresql.org
Cc: pryzby@telsasoft.com, andres@anarazel.de, tgl@sss.pgh.pa.us, robertmhaas@gmail.com, andrew@dunslane.net, thomas.munro@gmail.com, melanieplageman@gmail.com, peter.eisentraut@enterprisedb.com, daniel@yesql.se
Subject: Re: Adding CI to our tree
In-Reply-To: <20220227031057.GD25269@telsasoft.com>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox