Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl) - Mailing list pgsql-hackers

From Andrew Dunstan
Subject Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)
Date
Msg-id b1b1102c-1569-4406-a2c8-7947af9ef395@dunslane.net
Whole thread Raw
In response to Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)  (Noah Misch <noah@leadboat.com>)
Responses Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)
List pgsql-hackers
On 2025-04-12 Sa 10:10 PM, Noah Misch wrote:
> On Sat, Apr 12, 2025 at 07:51:06PM +0200, Wolfgang Walther wrote:
>> With injection points enabled, I get the following errors in test_aio:
>>
>>
>> [15:14:45.408](0.000s) not ok 187 - worker: first hard IO error is reported:
>> expected stderr
>> [15:14:45.409](0.000s)
>> [15:14:45.409](0.000s) #   Failed test 'worker: first hard IO error is
>> reported: expected stderr'
>> #   at t/001_aio.pl line 810.
>> [15:14:45.409](0.000s) #                   'psql:<stdin>:88: ERROR:  could
>> not read blocks 2..2 in file "base/5/16408": I/O error'
>> #     doesn't match '(?^:ERROR:.*could not read blocks 2\.\.2 in file
>> \"base/.*\": Input/output error)'
>> It seems like it's just the error message that is different and has "I/O"
>> instead of "Input/output"?
> Looks like it.
>
>> On a more general note, does enabling injection points make any sense here?
> Yes, it does.
>
>> I see that coverage in the build farm is not very big. IIUC, those are a
>> development tool, so might not be relevant, because nobody is developing on
>> Alpine / musl?
> No, whether anyone develops on the platform is not a factor.  One hasn't fully
> tested PostgreSQL until one builds with injection points.
>
>

Here's a simple fix ... also removes some unnecessary escaping and 
leaning toothpick syndrome.


cheers


andrew


--
Andrew Dunstan
EDB: https://www.enterprisedb.com

Attachment

pgsql-hackers by date:

Previous
From: Andrew Dunstan
Date:
Subject: Re: remove unnecessary explicit type conversion (to char) for appendStringInfoChar function calls
Next
From: Noah Misch
Date:
Subject: Re: Back-patch of: avoid multiple hard links to same WAL file after a crash