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

From: Wolfgang Walther <walther(at)technowledgy(dot)de>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>, Noah Misch <noah(at)leadboat(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Buildfarm: Enabling injection points on basilisk/dogfish (Alpine / musl)
Date: 2025-04-13 17:27:33
Message-ID: 67de1396-ed47-4703-bde2-a6623332c60b@technowledgy.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Andrew Dunstan:
>
> 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.

Confirmed - this works!

Thanks,

Wolfgang

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jose Luis Tallon 2025-04-13 17:45:40 Re: Automatically sizing the IO worker pool
Previous Message Abhishek Chanda 2025-04-13 16:40:29 Re: Adding error messages to a few slash commands