Roles for pg_basebackup

Started by Daniel Gustafssonalmost 6 years ago7 messagesdocs
Jump to latest
#1Daniel Gustafsson
daniel@yesql.se

Based on a recent conversation about backups I had I propose a small tweak to
the pg_basebackup documentation. Listing the user types in the reverse order
from today, putting superuser last, makes it IMO a little clearer that a
REPLICATION role is preferrable to using a superuser for running backups.

cheers ./daniel

Attachments:

pg_basebackup_user.diffapplication/octet-stream; name=pg_basebackup_user.diff; x-unix-mode=0644Download+2-2
#2Michael Paquier
michael@paquier.xyz
In reply to: Daniel Gustafsson (#1)
Re: Roles for pg_basebackup

On Mon, Apr 27, 2020 at 12:16:41PM +0200, Daniel Gustafsson wrote:

Based on a recent conversation about backups I had I propose a small tweak to
the pg_basebackup documentation. Listing the user types in the reverse order
from today, putting superuser last, makes it IMO a little clearer that a
REPLICATION role is preferrable to using a superuser for running backups.

Makes sense to me. We do that in logical-replication.sgml as well as
pg_rewind.sgml (the latter outlines superuser rights last).

Any objections from others?
--
Michael

#3Fujii Masao
masao.fujii@gmail.com
In reply to: Michael Paquier (#2)
Re: Roles for pg_basebackup

On 2020/04/28 13:37, Michael Paquier wrote:

On Mon, Apr 27, 2020 at 12:16:41PM +0200, Daniel Gustafsson wrote:

Based on a recent conversation about backups I had I propose a small tweak to
the pg_basebackup documentation. Listing the user types in the reverse order
from today, putting superuser last, makes it IMO a little clearer that a
REPLICATION role is preferrable to using a superuser for running backups.

Makes sense to me. We do that in logical-replication.sgml as well as
pg_rewind.sgml (the latter outlines superuser rights last).

Seems there are other documentations having the similar description,
for example, pg_receivewal.sgml, func.sgml and high-availability.sgml.
Isn't it better to update also them at the same time?

Any objections from others?

No.

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

#4Daniel Gustafsson
daniel@yesql.se
In reply to: Fujii Masao (#3)
Re: Roles for pg_basebackup

On 28 Apr 2020, at 07:22, Fujii Masao <masao.fujii@oss.nttdata.com> wrote:

On 2020/04/28 13:37, Michael Paquier wrote:

On Mon, Apr 27, 2020 at 12:16:41PM +0200, Daniel Gustafsson wrote:

Based on a recent conversation about backups I had I propose a small tweak to
the pg_basebackup documentation. Listing the user types in the reverse order
from today, putting superuser last, makes it IMO a little clearer that a
REPLICATION role is preferrable to using a superuser for running backups.

Makes sense to me. We do that in logical-replication.sgml as well as
pg_rewind.sgml (the latter outlines superuser rights last).

Seems there are other documentations having the similar description,
for example, pg_receivewal.sgml, func.sgml and high-availability.sgml.
Isn't it better to update also them at the same time?

That doe make sense. I've updated the places where I think a user configuring
the system might conceivably infer a preference from the order, but have left
the other ones as is.

cheers ./daniel

Attachments:

pg_basebackup_user-v2.diffapplication/octet-stream; name=pg_basebackup_user-v2.diff; x-unix-mode=0644Download+7-7
#5Fujii Masao
masao.fujii@gmail.com
In reply to: Daniel Gustafsson (#4)
Re: Roles for pg_basebackup

On 2020/04/28 17:15, Daniel Gustafsson wrote:

On 28 Apr 2020, at 07:22, Fujii Masao <masao.fujii@oss.nttdata.com> wrote:

On 2020/04/28 13:37, Michael Paquier wrote:

On Mon, Apr 27, 2020 at 12:16:41PM +0200, Daniel Gustafsson wrote:

Based on a recent conversation about backups I had I propose a small tweak to
the pg_basebackup documentation. Listing the user types in the reverse order
from today, putting superuser last, makes it IMO a little clearer that a
REPLICATION role is preferrable to using a superuser for running backups.

Makes sense to me. We do that in logical-replication.sgml as well as
pg_rewind.sgml (the latter outlines superuser rights last).

Seems there are other documentations having the similar description,
for example, pg_receivewal.sgml, func.sgml and high-availability.sgml.
Isn't it better to update also them at the same time?

That doe make sense. I've updated the places where I think a user configuring
the system might conceivably infer a preference from the order, but have left
the other ones as is.

Thanks for updating the patch! It looks good to me.

Regards,

--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION

#6Michael Paquier
michael@paquier.xyz
In reply to: Fujii Masao (#5)
Re: Roles for pg_basebackup

On Fri, May 01, 2020 at 12:29:09PM +0900, Fujii Masao wrote:

On 2020/04/28 17:15, Daniel Gustafsson wrote:

That does make sense. I've updated the places where I think a user configuring
the system might conceivably infer a preference from the order, but have left
the other ones as is.

Yeah, I spotted a couple of places in adminpack.sgml, dropuser.sgml,
catalogs.sgml and config.sgml, but the intention looks different. We
could always discuss more on that later on.

Thanks for updating the patch! It looks good to me.

Fine for me as well as far as it goes. I'll go apply it if there are
no objections.
--
Michael

#7Michael Paquier
michael@paquier.xyz
In reply to: Michael Paquier (#6)
Re: Roles for pg_basebackup

On Sun, May 03, 2020 at 03:53:31PM +0900, Michael Paquier wrote:

Fine for me as well as far as it goes. I'll go apply it if there are
no objections.

And done.
--
Michael