From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org, Michael Paquier <michael(at)paquier(dot)xyz> |
Subject: | Re: pgsql: Support "postgres -C" with runtime-computed GUCs |
Date: | 2021-09-21 04:32:09 |
Message-ID: | 216D2155-018C-4FFB-8FE3-73CAA5A3FEE1@anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Hi,
On September 20, 2021 8:47:24 PM PDT, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>On Mon, Sep 20, 2021 at 08:20:40PM -0700, Andres Freund wrote:
>> So the problem is that -C data_checksums requires permissions to be dropped,
>> but the way it's invoked here that's not guaranteed... And I don't think we
>> should remove that requirement, given that we're actually doing stuff to the
>> lock file...
>
>Hmm. One way I can think of to keep the test would be something like
>that to avoid problems with pg_ctl redirecting the server's stderr to
>stdout (hence the tweak for log_min_messages):
>pg_ctl start -s -o '-C data_checksums -c log_min_messages=fatal'
Yes, that should work. Although error should suffice.
FWIW, I think it'd be good to hide that message in the -C case.
>This would print the parameter, while getting to stderr mostly a
>"could not start server". Would that work for you?
Yes, I think so.
Andres
--
Sent from my Android device with K-9 Mail. Please excuse my brevity.
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2021-09-21 04:42:34 | Re: pgsql: Support "postgres -C" with runtime-computed GUCs |
Previous Message | Michael Paquier | 2021-09-21 03:47:24 | Re: pgsql: Support "postgres -C" with runtime-computed GUCs |