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 |
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 |