Re: pg_regress cleans up tablespace twice.

From: Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: pg_regress cleans up tablespace twice.
Date: 2020-06-20 03:01:36
Message-ID: CA+hUKGJ79vRQ+OP7MKM8R8uncf=fGCi3SWGwhZx_GteRCv7MWw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Jun 20, 2020 at 2:42 PM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
> > +ERROR: could not set permissions on directory
> > "C:/projects/postgresql/src/test/regress/testtablespace": Permission
> > denied
> >
> > Any ideas? Here's what it does:
> >
> > https://github.com/macdice/cfbot/tree/master/appveyor
>
> I am not sure, and I am not really familiar with this stuff. Your
> code does a simple vcregress check, and that should take care of
> automatically cleaning up the testtablespace path. The buildfarm uses
> this code for MSVC builds and does not complain, nor do my own VMs
> complain. A difference in the processing after 2b2a070d is that the
> tablespace cleanup/creation does not happen while holding a restricted
> token [1] anymore because it got out of pg_regress.c. Are there any
> kind of restrictions applied to the user running appveyor on Windows?

Thanks for the clue. Appveyor runs your build script as a privileged
user (unlike, I assume, the build farm animals), and that has caused a
problem with this test in the past, though I don't know the details.
I might go and teach it to skip that test until a fix can be found.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Amit Kapila 2020-06-20 04:15:52 Re: min_safe_lsn column in pg_replication_slots view
Previous Message Michael Paquier 2020-06-20 02:48:09 Re: Failures with installcheck and low work_mem value in 13~