pg.ddx.io pgsql-bugs@postgresql.org mailing list archive
help / color / mirror / Atom feedBUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
5+ messages / 4 participants
[nested] [flat]
* BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
@ 2024-09-11 06:42 PG Bug reporting form <noreply@postgresql.org>
0 siblings, 1 reply; 5+ messages in thread
From: PG Bug reporting form @ 2024-09-11 06:42 UTC (permalink / raw)
To: pgsql-bugs@lists.postgresql.org; +Cc: n.kalinin@postgrespro.ru
The following bug has been logged on the website:
Bug reference: 18609
Logged by: Nikita Kalinin
Email address: n.kalinin@postgrespro.ru
PostgreSQL version: 16.4
Operating system: ubuntu 22.04
Description:
Hello!
When running the installcheck a second time, the test_pg_dump fails due to
the following reason:
CREATE ROLE regress_dump_test_role;
+ERROR: role "regress_dump_test_role" already exists
CREATE EXTENSION test_pg_dump;
This issue is present from REL_12_STABLE to REL_16_STABLE. Maybe it's worth
adding DROP ROLE regress_dump_test_role at the end of the test, like in
REL_17_STABLE?
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
@ 2024-09-11 08:01 Daniel Gustafsson <daniel@yesql.se>
parent: PG Bug reporting form <noreply@postgresql.org>
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Gustafsson @ 2024-09-11 08:01 UTC (permalink / raw)
To: n.kalinin@postgrespro.ru; pgsql-bugs@lists.postgresql.org
> On 11 Sep 2024, at 08:42, PG Bug reporting form <noreply@postgresql.org> wrote:
>
> The following bug has been logged on the website:
>
> Bug reference: 18609
> Logged by: Nikita Kalinin
> Email address: n.kalinin@postgrespro.ru
> PostgreSQL version: 16.4
> Operating system: ubuntu 22.04
> Description:
>
> Hello!
> When running the installcheck a second time, the test_pg_dump fails due to
> the following reason:
> CREATE ROLE regress_dump_test_role;
> +ERROR: role "regress_dump_test_role" already exists
> CREATE EXTENSION test_pg_dump;
>
> This issue is present from REL_12_STABLE to REL_16_STABLE. Maybe it's worth
> adding DROP ROLE regress_dump_test_role at the end of the test, like in
> REL_17_STABLE?
That would be backpatching 936e3fa3787a51397280c1081587586e83c20399. I haven't
checked if there are complications in doing so but I think it should be quite
straightforward.
--
Daniel Gustafsson
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
@ 2024-09-19 07:44 Michael Paquier <michael@paquier.xyz>
parent: Daniel Gustafsson <daniel@yesql.se>
0 siblings, 1 reply; 5+ messages in thread
From: Michael Paquier @ 2024-09-19 07:44 UTC (permalink / raw)
To: Daniel Gustafsson <daniel@yesql.se>; +Cc: n.kalinin@postgrespro.ru; pgsql-bugs@lists.postgresql.org
On Wed, Sep 11, 2024 at 10:01:22AM +0200, Daniel Gustafsson wrote:
> That would be backpatching 936e3fa3787a51397280c1081587586e83c20399. I haven't
> checked if there are complications in doing so but I think it should be quite
> straightforward.
As far as I can see it would be enough to add the DROP OWNED BY to
avoid the random notice messages listing the dependencies the role
depends on, followed by the DROP ROLE. FWIW, I am not seeing any
complications (the TAP test recreates the same role name
regress_dump_test_role on its own), so let's just do it.
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
@ 2024-09-23 08:21 Никита Калинин <n.kalinin@postgrespro.ru>
parent: Michael Paquier <michael@paquier.xyz>
0 siblings, 1 reply; 5+ messages in thread
From: Никита Калинин @ 2024-09-23 08:21 UTC (permalink / raw)
To: Michael Paquier <michael@paquier.xyz>; pgsql-bugs@lists.postgresql.org
Yes, it will be enough to add DROP OWNED BY.
Perhaps with these changes, we can allow meson builds to run the test_pg_dump test in installcheck mode on the REL_16_STABLE branch?
> On 19 Sep 2024, at 14:44, Michael Paquier <michael@paquier.xyz> wrote:
>
> DROP OWNED BY
^ permalink raw reply [nested|flat] 5+ messages in thread
* Re: BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role
@ 2024-09-24 03:41 Michael Paquier <michael@paquier.xyz>
parent: Никита Калинин <n.kalinin@postgrespro.ru>
0 siblings, 0 replies; 5+ messages in thread
From: Michael Paquier @ 2024-09-24 03:41 UTC (permalink / raw)
To: Никита Калинин <n.kalinin@postgrespro.ru>; +Cc: pgsql-bugs@lists.postgresql.org
On Mon, Sep 23, 2024 at 03:21:13PM +0700, Никита Калинин wrote:
> Yes, it will be enough to add DROP OWNED BY.
> Perhaps with these changes, we can allow meson builds to run the
> test_pg_dump test in installcheck mode on the REL_16_STABLE branch?
Yes, like b0c5b215dace, we should. REL_17_STABLE is frozen this week
for the new major release, but the other branches are not, so I have
applied a cherry-pick of 936e3fa3787a to 12~16, and b0c5b215dace to
16.
--
Michael
Attachments:
signature.asc (application/pgp-signature, 832B) download
^ permalink raw reply [nested|flat] 5+ messages in thread
end of thread, other threads:[~2024-09-24 03:41 UTC | newest]
Thread overview: 5+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2024-09-11 06:42 BUG #18609: Repeated installcheck failure in test_pg_dump due to existing role PG Bug reporting form <noreply@postgresql.org>
2024-09-11 08:01 ` Daniel Gustafsson <daniel@yesql.se>
2024-09-19 07:44 ` Michael Paquier <michael@paquier.xyz>
2024-09-23 08:21 ` Никита Калинин <n.kalinin@postgrespro.ru>
2024-09-24 03:41 ` Michael Paquier <michael@paquier.xyz>
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox