Re: pgsql: Add TAP tests for contrib/sslinfo

From: Daniel Gustafsson <daniel(at)yesql(dot)se>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Add TAP tests for contrib/sslinfo
Date: 2021-11-30 16:58:05
Message-ID: FB603D1D-2F48-48E1-B1BB-4DEF95DAF393@yesql.se
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

> On 30 Nov 2021, at 17:40, Daniel Gustafsson <daniel(at)yesql(dot)se> wrote:
>
>> On 30 Nov 2021, at 11:50, Daniel Gustafsson <dgustafsson(at)postgresql(dot)org> wrote:
>>
>> Add TAP tests for contrib/sslinfo
>
> This just failed on fairywren with the below:
>
> not ok 1 - certificate authorization succeeds with correct client cert in PEM format
>
> # Failed test 'certificate authorization succeeds with correct client cert in PEM format'
> # at t/003_sslinfo.pl line 71.
> # got: '2'
> # expected: '0'
> connection error: 'psql: error: connection to server at "127.0.0.1", port 60695 failed: certificate present, but not private key file "/home/pgrunner/bf/root/HEAD/pgsql.build/src/test/ssl/tmp_check/client_ext.key"'
> while running 'psql -XAtq -d sslrootcert=ssl/root+server_ca.crt sslmode=require dbname=certdb hostaddr=127.0.0.1 user=ssltestuser sslcert=ssl/client_ext.crt sslkey=/home/pgrunner/bf/root/HEAD/pgsql.build/src/test/ssl/tmp_check/client_ext.key -f - -v ON_ERROR_STOP=1' at /home/pgrunner/bf/root/HEAD/pgsql.build/../pgsql/src/test/perl/PostgreSQL/Test/Cluster.pm line 1807.
>
> ..which is odd, since the key was copied to tmp_check earlier in the test and
> passed that. The 001 and 002 tests do the same kind of copying and there it
> worked, so it seems a tad odd. Does anyone have any ideas on insights here?

Scratch that, all the copying for tests 001 through 003 had failed. I clearly
need another coffee.

The question still stands though, does anyone have any ideas on what could've
happened as I'm currently drawing a blank?

--
Daniel Gustafsson https://vmware.com/

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Alvaro Herrera 2021-11-30 18:42:06 pgsql: Increase size of shared memory for pg_commit_ts
Previous Message Daniel Gustafsson 2021-11-30 16:40:41 Re: pgsql: Add TAP tests for contrib/sslinfo