Re: Adding null patch entry to cfbot/CommitFest - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Adding null patch entry to cfbot/CommitFest
Date
Msg-id 605772.1747710693@sss.pgh.pa.us
Whole thread Raw
In response to Adding null patch entry to cfbot/CommitFest  (Tatsuo Ishii <ishii@postgresql.org>)
Responses Re: Adding null patch entry to cfbot/CommitFest
List pgsql-hackers
"David G. Johnston" <david.g.johnston@gmail.com> writes:
> On Monday, May 19, 2025, Tatsuo Ishii <ishii@postgresql.org> wrote:
>> I have observed cases where a cfbot entry fails without clear reason
>> [1]. Even if the patch just modifies comments, it has failed in some
>> cfbot test. In this case we could easily guess that master branch
>> might have problems at the time when the tests performed.

> I too made this realization while reviewing the application.  I concur it
> is something that we should try and mitigate.  Sending a canary patch
> through once-a-day, or on any fixed time period, doesn’t quite seem
> sufficient.

Yeah, I'm afraid that won't do much except eat valuable cycles.

IME the real problem with the CI tests is that the CI infrastructure
itself is not very stable, and intermittently fails tests for reasons
having little to do with the tested patch *or* the state of the master
branch.  If there is a problem in master, we typically know it because
the buildfarm is also unhappy, so I'm not clear on what a null patch
in the queue would add.

            regards, tom lane



pgsql-hackers by date:

Previous
From: shveta malik
Date:
Subject: Re: Conflict detection for update_deleted in logical replication
Next
From: Michael Paquier
Date:
Subject: Re: Add comment explaining why queryid is int64 in pg_stat_statements