github/repmgr
1
0
Fork 0
mirror of https://github.com/EnterpriseDB/repmgr.git synced 2026-03-27 17:06:29 +00:00
Code Issues Packages Projects Releases 1 Wiki Activity

Improve "archive_mode" configuration check

Browse Source 
There's no compelling reason to require "archive_mode" to be enabled
for streaming replication. It is of course a good idea to archive WAL
using e.g. barman ( http://www.pgbarman.org/ ) as part of a comprehensive
backup strategy, but repmgr and streaming replication work fine without
it.

Per GitHub #141.

Also revise the configuration check for "archive_command" to be
triggered only when "archive_mode" is not "off", as from PostgreSQL
9.5 onwards "archive_mode" can also be "on" or "always".
This commit is contained in:
Ian Barwick
2016-01-14 09:33:08 +09:00
parent 7e6bac1be6
commit f3d0ab9ab9
1 changed files with 12 additions and 23 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
repmgr.c
View File

@@ -4314,31 +4314,18 @@ check_upstream_config(PGconn *conn, int server_version_num, bool exit_on_error)
} }
} }
i = guc_set(conn, "archive_mode", "=", "on");
if (i == 0 || i == -1)
{
if (i == 0)
log_err(_("parameter 'archive_mode' must be set to 'on'\n"));
if (exit_on_error == true)
{
PQfinish(conn);
exit(ERR_BAD_CONFIG);
}
config_ok = false;
}
/* /*
* check that 'archive_command' is non empty (however it's not practical to * If archive_mode is enabled, check that 'archive_command' is non empty
* check that it's actually valid) * (however it's not practical to check that it actually represents a valid
* command).
* *
* if 'archive_mode' is not on, pg_settings returns '(disabled)' regardless * From PostgreSQL 9.5, archive_mode can be one of 'off', 'on' or 'always'
* of what's in 'archive_command', so until 'archive_mode' is on we can't * so for ease of backwards compatibility, rather than explicitly check for an
* properly check it. * enabled mode, check that it's not "off".
*/ */
if (guc_set(conn, "archive_mode", "=", "on"))
if (guc_set(conn, "archive_mode", "!=", "off"))
{ {
i = guc_set(conn, "archive_command", "!=", ""); i = guc_set(conn, "archive_command", "!=", "");
@@ -4360,9 +4347,11 @@ check_upstream_config(PGconn *conn, int server_version_num, bool exit_on_error)
/* /*
* Check that 'hot_standby' is on. This isn't strictly necessary * Check that 'hot_standby' is on. This isn't strictly necessary
* for the primary server, however the assumption is that configuration * for the primary server, however the assumption is that we'll be
* should be consistent for all servers in a cluster. * cloning standbys and thus copying the primary configuration;
* this way the standby will be correctly configured by default.
*/ */
i = guc_set(conn, "hot_standby", "=", "on"); i = guc_set(conn, "hot_standby", "=", "on");
if (i == 0 || i == -1) if (i == 0 || i == -1)
{ {