Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: Noah Misch <noah(at)leadboat(dot)com>, Wolfgang Walther <walther(at)technowledgy(dot)de>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)
Date: 2025-04-13 15:19:29
Message-ID: b1b1102c-1569-4406-a2c8-7947af9ef395@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: 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 Content-Type Size
aio-regex-fix.patch text/x-patch 955 bytes

In response to

Responses

Browse pgsql-hackers by date

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