Re: Allow non-superuser to cancel superuser tasks.

From: Kirill Reshke <reshkekirill(at)gmail(dot)com>
To: Nathan Bossart <nathandbossart(at)gmail(dot)com>
Cc: "Andrey M(dot) Borodin" <x4mmm(at)yandex-team(dot)ru>, Michael Paquier <michael(at)paquier(dot)xyz>, "Leung, Anthony" <antholeu(at)amazon(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Allow non-superuser to cancel superuser tasks.
Date: 2024-07-10 05:03:04
Message-ID: CALdSSPhsVDhVqEW63FsDtRsaL=KmrVV-0RZYE3EjiWe9h6MyeA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi

On Tue, 9 Jul 2024 at 23:13, Nathan Bossart <nathandbossart(at)gmail(dot)com> wrote:
>
> I've committed 0001. It looks like 0002 failed CI testing [0], but I
> haven't investigated why.
>
> [0] https://cirrus-ci.com/task/5668467599212544
>
> --
> nathan

The problem is the error message has been changed.

# DETAIL: Only roles with privileges of the
"pg_signal_autovacuum_worker" role may terminate autovacuum workers.'
# doesn't match '(?^:ERROR: permission denied to terminate
process\nDETAIL: Only roles with privileges of the
"pg_signal_autovacuum_worker" role may terminate autovacuum worker
processes.)'
# Looks like you failed 1 test of 2.

I changed the test to match the error message.

Attachment Content-Type Size
v7-0002-Add-tap-test-for-pg_signal_autovacuum-role.patch application/octet-stream 4.6 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2024-07-10 05:14:55 Re: Allow non-superuser to cancel superuser tasks.
Previous Message Kartyshov Ivan 2024-07-10 04:58:23 Re: [HACKERS] make async slave to wait for lsn to be replayed