Re: Weird test mixup

On 15/03/2024 13:09, Heikki Linnakangas wrote:
> On 15/03/2024 01:13, Tom Lane wrote:
>> Michael Paquier <michael(at)paquier(dot)xyz> writes:
>>> Or we could just disable runningcheck because of the concurrency
>>> requirement in this test. The test would still be able to run, just
>>> less times.
>>
>> No, actually we *must* mark all these tests NO_INSTALLCHECK if we
>> stick with the current definition of injection points. The point
>> of installcheck mode is that the tests are supposed to be safe to
>> run in a live installation. Side-effects occurring in other
>> databases are completely not OK.
>
> I committed a patch to do that, to put out the fire.

That's turning the buildfarm quite red. Many, but not all animals are
failing like this:

> --- /home/buildfarm/hippopotamus/buildroot/HEAD/pgsql.build/src/test/modules/injection_points/expected/injection_points.out 2024-03-15 12:41:16.363286975 +0100
> +++ /home/buildfarm/hippopotamus/buildroot/HEAD/pgsql.build/src/test/modules/injection_points/results/injection_points.out 2024-03-15 12:53:11.528159615 +0100
> @@ -1,118 +1,111 @@
> CREATE EXTENSION injection_points;
> +ERROR: extension "injection_points" is not available
> +DETAIL: Could not open extension control file "/home/buildfarm/hippopotamus/buildroot/HEAD/pgsql.build/tmp_install/home/buildfarm/hippopotamus/buildroot/HEAD/inst/share/postgresql/extension/injection_points.control": No such file or directory.
> +HINT: The extension must first be installed on the system where PostgreSQL is running.
> ...

Looks like adding NO_INSTALLCHECK somehow affected how the modules are
installed in tmp_install. I'll investigate..

--
Heikki Linnakangas
Neon (https://neon.tech)