Re: pgsql: Provide a TLS init hook

From: Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Provide a TLS init hook
Date: 2020-03-26 15:11:08
Message-ID: 6965f28d-8ff4-a371-dacf-ce21349d52c5@2ndQuadrant.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers


On 3/26/20 9:50 AM, Tom Lane wrote:
> Andrew Dunstan <andrew(dot)dunstan(at)2ndquadrant(dot)com> writes:
>> On 3/25/20 9:28 PM, Tom Lane wrote:
>>> jacana has just exposed a different problem: it's not configured
>>> --with-openssl, but the buildfarm script is trying to run this
>>> new test module anyway. I'm confused about the reason.
>>> "make installcheck" in src/test/modules does the right thing,
>>> but seemingly that client is doing something different?
>> Ugh. I have put in place a hack to clear the error on jacana. Yes, the
>> client does something different so we can run each module separately.
>> Trawling through the output and files for one test on its own is hard
>> enough, I don't want to aggregate them.
> Well, I'm confused, because my own critters are running this as part
> of a single make-installcheck-in-src/test/modules step, eg
>
> https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=longfin&dt=2020-03-26%2002%3A09%3A08&stg=testmodules-install-check-C
>
> Why is jacana doing it differently?

longfin is also running it (first) here
https://buildfarm.postgresql.org/cgi-bin/show_stage_log.pl?nm=longfin&dt=2020-03-26%2014%3A39%3A51&stg=ssl_passphrase_callback-check

That's where jacana failed.

I don't think this belongs in installcheck, we should add
'NO_INSTALLCHECK = 1' to the Makefile.

cheers

andrew

--
Andrew Dunstan https://www.2ndQuadrant.com
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Andrew Dunstan 2020-03-26 15:15:32 Re: pgsql: Revert "initdb: Change authentication defaults"
Previous Message Magnus Hagander 2020-03-26 14:06:36 pgsql: Document that pg_checksums exists in checksums README

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2020-03-26 15:31:13 Re: pgsql: Provide a TLS init hook
Previous Message Justin Pryzby 2020-03-26 15:04:57 Re: error context for vacuum to include block number