5f47b2ef11
- update to version 2.46 * Features: + Block incremental backup. (Reviewed by John Morris, Stephen Frost, Stefan Fercot.) + SFTP support for repository storage. (Contributed by Reid Thompson. Reviewed by Stephen Frost, David Steele.) + PostgreSQL 16 support. (Reviewed by Stefan Fercot.) * Improvements: + Allow page header checks to be skipped. (Reviewed by David Christensen. Suggested by David Christensen.) + Avoid chown() on recovery files during restore. (Reviewed by Stefan Fercot, Marcelo Henrique Neppel. Suggested by Marcelo Henrique Neppel.) + Add error retry detail for HTTP retries. * Documentation Improvements: + Add warning about using recovery type=none. (Reviewed by Stefan Fercot.) + Add note about running stanza-create on already-created repositories. OBS-URL: https://build.opensuse.org/request/show/1089458 OBS-URL: https://build.opensuse.org/package/show/server:database:postgresql/pgbackrest?expand=0&rev=57
162 lines
15 KiB
Diff
162 lines
15 KiB
Diff
diff --color -ur pgbackrest-release-2.46.orig/CODING.md pgbackrest-release-2.46/CODING.md
|
|
--- pgbackrest-release-2.46.orig/CODING.md 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/CODING.md 2023-05-28 21:20:39.162416107 +0200
|
|
@@ -268,11 +268,11 @@
|
|
|
|
This function can be called without variable parameters:
|
|
```c
|
|
-storagePathCreateP(storageLocal(), "/tmp/pgbackrest");
|
|
+storagePathCreateP(storageLocal(), "/run/pgbackrest");
|
|
```
|
|
Or with variable parameters:
|
|
```c
|
|
-storagePathCreateP(storageLocal(), "/tmp/pgbackrest", .errorOnExists = true, .mode = 0777);
|
|
+storagePathCreateP(storageLocal(), "/run/pgbackrest", .errorOnExists = true, .mode = 0777);
|
|
```
|
|
If the majority of functions in a module or object are variadic it is best to provide macros for all functions even if they do not have variable parameters. Do not use the base function when variadic macros exist.
|
|
|
|
diff --color -ur pgbackrest-release-2.46.orig/doc/resource/git-history.cache pgbackrest-release-2.46/doc/resource/git-history.cache
|
|
--- pgbackrest-release-2.46.orig/doc/resource/git-history.cache 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/doc/resource/git-history.cache 2023-05-28 21:20:02.622040915 +0200
|
|
@@ -6567,7 +6567,7 @@
|
|
{
|
|
"commit": "a1365b26d4c2383eaacea3714eeb521435ca79aa",
|
|
"date": "2021-01-18 23:41:09 -0500",
|
|
- "subject": "Remove duplicate mkdir for /tmp/pgbackrest in Vagrantfile."
|
|
+ "subject": "Remove duplicate mkdir for /run/pgbackrest in Vagrantfile."
|
|
},
|
|
{
|
|
"commit": "8322dfd7af97e307b952cfb37e6a65b65f55be0f",
|
|
@@ -9383,8 +9383,8 @@
|
|
{
|
|
"commit": "3fbfcba811499a59b943dd893eef9ff52090f7b6",
|
|
"date": "2020-03-16 17:27:01 -0400",
|
|
- "subject": "Forbid access to /tmp/pgbackrest in the Vagrantfile.",
|
|
- "body": "This matches the error that will be thrown in the vm=none test on Travis CI if a unit test writes to /tmp/pgbackrest."
|
|
+ "subject": "Forbid access to /run/pgbackrest in the Vagrantfile.",
|
|
+ "body": "This matches the error that will be thrown in the vm=none test on Travis CI if a unit test writes to /run/pgbackrest."
|
|
},
|
|
{
|
|
"commit": "46911c64c19dc8ff76d2a6ff4548fc01647043e0",
|
|
@@ -11284,7 +11284,7 @@
|
|
"commit": "45881c74aeff4bb25559ec0254fa7fc1960d9cab",
|
|
"date": "2019-10-08 12:06:30 -0400",
|
|
"subject": "Allow most unit tests to run outside of a container.",
|
|
- "body": "Three major changes were required to get this working:\n\n1) Provide the path to pgbackrest in the build directory when running outside a container. Tests in a container will continue to install and run against /usr/bin/pgbackrest.\n\n1) Set a per-test lock path so tests don't conflict on the default /tmp/pgbackrest path. Also set a per-test log-path while we are at it.\n\n2) Use localhost instead of a custom host for TLS test connections. Tests in containers will continue to update /etc/hosts and use the custom host.\n\nAdd infrastructure and update harnessCfgLoad*() to get the correct exe and paths loaded for testing.\n\nSince new tests are required to verify that running outside a container works, also rework the tests in Travis CI to provide coverage within a reasonable amount of time. Mainly, break up to doc tests by VM and run an abbreviated unit test suite on co6 and co7."
|
|
+ "body": "Three major changes were required to get this working:\n\n1) Provide the path to pgbackrest in the build directory when running outside a container. Tests in a container will continue to install and run against /usr/bin/pgbackrest.\n\n1) Set a per-test lock path so tests don't conflict on the default /run/pgbackrest path. Also set a per-test log-path while we are at it.\n\n2) Use localhost instead of a custom host for TLS test connections. Tests in containers will continue to update /etc/hosts and use the custom host.\n\nAdd infrastructure and update harnessCfgLoad*() to get the correct exe and paths loaded for testing.\n\nSince new tests are required to verify that running outside a container works, also rework the tests in Travis CI to provide coverage within a reasonable amount of time. Mainly, break up to doc tests by VM and run an abbreviated unit test suite on co6 and co7."
|
|
},
|
|
{
|
|
"commit": "77b0c6c993a0e6ff45a6a99f343c3709a016d152",
|
|
@@ -19771,7 +19771,7 @@
|
|
"commit": "18fd25233b1db7b3197f5d8dc537d239dcfc7aba",
|
|
"date": "2016-04-14 09:30:54 -0400",
|
|
"subject": "New simpler configuration and consistent project/exe/path naming.",
|
|
- "body": "* The repo-path option now always refers to the repository where backups and archive are stored, whether local or remote, so the repo-remote-path option has been removed. The new spool-path option can be used to define a location for queueing WAL segments when archiving asynchronously. Otherwise, a local repository is no longer required.\n\n* Implemented a new config format which should be far simpler to use. See the User Guide and Configuration Reference for details but for a simple configuration all options can now be placed in the stanza section. Options that are shared between stanzas can be placed in the [global] section. More complex configurations can still make use of command sections though this should be a rare use case.\n\n* The default configuration filename is now pgbackrest.conf instead of pg_backrest.conf. This was done for consistency with other naming changes but also to prevent old config files from being loaded accidentally.\n\n* The default repository name was changed from /var/lib/backup to /var/lib/pgbackrest.\n\n* Lock files are now stored in /tmp/pgbackrest by default. These days /run/pgbackrest would be the preferred location but that would require init scripts which are not part of this release. The lock-path option can be used to configure the lock directory.\n\n* Log files are now stored in /var/log/pgbackrest by default and no longer have the date appended so they can be managed with logrotate. The log-path option can be used to configure the lock directory.\n\n* Executable filename changed from pg_backrest to pgbackrest."
|
|
+ "body": "* The repo-path option now always refers to the repository where backups and archive are stored, whether local or remote, so the repo-remote-path option has been removed. The new spool-path option can be used to define a location for queueing WAL segments when archiving asynchronously. Otherwise, a local repository is no longer required.\n\n* Implemented a new config format which should be far simpler to use. See the User Guide and Configuration Reference for details but for a simple configuration all options can now be placed in the stanza section. Options that are shared between stanzas can be placed in the [global] section. More complex configurations can still make use of command sections though this should be a rare use case.\n\n* The default configuration filename is now pgbackrest.conf instead of pg_backrest.conf. This was done for consistency with other naming changes but also to prevent old config files from being loaded accidentally.\n\n* The default repository name was changed from /var/lib/backup to /var/lib/pgbackrest.\n\n* Lock files are now stored in /run/pgbackrest by default. These days /run/pgbackrest would be the preferred location but that would require init scripts which are not part of this release. The lock-path option can be used to configure the lock directory.\n\n* Log files are now stored in /var/log/pgbackrest by default and no longer have the date appended so they can be managed with logrotate. The log-path option can be used to configure the lock directory.\n\n* Executable filename changed from pg_backrest to pgbackrest."
|
|
},
|
|
{
|
|
"commit": "885797e4b58a675487a3531ba16908c1d1e9f970",
|
|
diff --color -ur pgbackrest-release-2.46.orig/doc/xml/coding.xml pgbackrest-release-2.46/doc/xml/coding.xml
|
|
--- pgbackrest-release-2.46.orig/doc/xml/coding.xml 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/doc/xml/coding.xml 2023-05-28 21:18:27.561053697 +0200
|
|
@@ -365,13 +365,13 @@
|
|
<p>This function can be called without variable parameters:</p>
|
|
|
|
<code-block type="c">
|
|
-storagePathCreateP(storageLocal(), "/tmp/pgbackrest");
|
|
+storagePathCreateP(storageLocal(), "/run/pgbackrest");
|
|
</code-block>
|
|
|
|
<p>Or with variable parameters:</p>
|
|
|
|
<code-block type="c">
|
|
-storagePathCreateP(storageLocal(), "/tmp/pgbackrest", .errorOnExists = true, .mode = 0777);
|
|
+storagePathCreateP(storageLocal(), "/run/pgbackrest", .errorOnExists = true, .mode = 0777);
|
|
</code-block>
|
|
|
|
<p>If the majority of functions in a module or object are variadic it is best to provide macros for all functions even if they do not have variable parameters. Do not use the base function when variadic macros exist.</p>
|
|
diff --color -ur pgbackrest-release-2.46.orig/doc/xml/release.xml pgbackrest-release-2.46/doc/xml/release.xml
|
|
--- pgbackrest-release-2.46.orig/doc/xml/release.xml 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/doc/xml/release.xml 2023-05-28 21:19:14.985551780 +0200
|
|
@@ -11136,7 +11136,7 @@
|
|
<release-item-contributor id="david.steele"/>
|
|
</release-item-contributor-list> -->
|
|
|
|
- <p>Lock files are now stored in <path>/tmp/pgbackrest</path> by default. These days <path>/run/pgbackrest</path> is the preferred location but that would require init scripts which are not part of this release. The <setting>lock-path</setting> option can be used to configure the lock directory.</p>
|
|
+ <p>Lock files are now stored in <path>/run/pgbackrest</path> by default. These days <path>/run/pgbackrest</path> is the preferred location but that would require init scripts which are not part of this release. The <setting>lock-path</setting> option can be used to configure the lock directory.</p>
|
|
</release-item>
|
|
|
|
<release-item>
|
|
Only in pgbackrest-release-2.46.orig/: patches
|
|
diff --color -ur pgbackrest-release-2.46.orig/src/build/config/config.yaml pgbackrest-release-2.46/src/build/config/config.yaml
|
|
--- pgbackrest-release-2.46.orig/src/build/config/config.yaml 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/src/build/config/config.yaml 2023-05-28 21:21:14.126750421 +0200
|
|
@@ -834,7 +834,7 @@
|
|
lock-path:
|
|
section: global
|
|
type: path
|
|
- default: /tmp/pgbackrest
|
|
+ default: /run/pgbackrest
|
|
command:
|
|
annotate: {}
|
|
archive-get: {}
|
|
diff --color -ur pgbackrest-release-2.46.orig/src/config/parse.auto.c.inc pgbackrest-release-2.46/src/config/parse.auto.c.inc
|
|
--- pgbackrest-release-2.46.orig/src/config/parse.auto.c.inc 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/src/config/parse.auto.c.inc 2023-05-28 21:22:00.283175880 +0200
|
|
@@ -11,7 +11,7 @@
|
|
|
|
static const StringPub parseRuleValueStr[] =
|
|
{
|
|
- PARSE_RULE_STRPUB("/tmp/pgbackrest"), // val/str
|
|
+ PARSE_RULE_STRPUB("/run/pgbackrest"), // val/str
|
|
PARSE_RULE_STRPUB("/var/lib/pgbackrest"), // val/str
|
|
PARSE_RULE_STRPUB("/var/log/pgbackrest"), // val/str
|
|
PARSE_RULE_STRPUB("/var/spool/pgbackrest"), // val/str
|
|
diff --color -ur pgbackrest-release-2.46.orig/test/ci.pl pgbackrest-release-2.46/test/ci.pl
|
|
--- pgbackrest-release-2.46.orig/test/ci.pl 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/test/ci.pl 2023-05-28 21:12:52.581888810 +0200
|
|
@@ -208,8 +208,8 @@
|
|
$strPackage .= " libdbd-pg-perl";
|
|
}
|
|
|
|
- processBegin('/tmp/pgbackrest owned by root so tests cannot use it');
|
|
- processExec('sudo mkdir -p /tmp/pgbackrest && sudo chown root:root /tmp/pgbackrest && sudo chmod 700 /tmp/pgbackrest');
|
|
+ processBegin('/run/pgbackrest owned by root so tests cannot use it');
|
|
+ processExec('sudo mkdir -p /run/pgbackrest && sudo chown root:root /run/pgbackrest && sudo chmod 700 /run/pgbackrest');
|
|
processEnd();
|
|
|
|
processBegin('install test packages');
|
|
diff --color -ur pgbackrest-release-2.46.orig/test/src/module/command/helpTest.c pgbackrest-release-2.46/test/src/module/command/helpTest.c
|
|
--- pgbackrest-release-2.46.orig/test/src/module/command/helpTest.c 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/test/src/module/command/helpTest.c 2023-05-28 21:20:59.282613590 +0200
|
|
@@ -244,7 +244,7 @@
|
|
" [default=n]\n"
|
|
" --io-timeout I/O timeout [default=60]\n"
|
|
" --lock-path path where lock files are stored\n"
|
|
- " [default=/tmp/pgbackrest]\n"
|
|
+ " [default=/run/pgbackrest]\n"
|
|
" --neutral-umask use a neutral umask [default=y]\n"
|
|
" --process-max max processes to use for\n"
|
|
" compress/transfer [default=1]\n"
|
|
diff --color -ur pgbackrest-release-2.46.orig/test/Vagrantfile pgbackrest-release-2.46/test/Vagrantfile
|
|
--- pgbackrest-release-2.46.orig/test/Vagrantfile 2023-05-22 10:13:13.000000000 +0200
|
|
+++ pgbackrest-release-2.46/test/Vagrantfile 2023-05-28 21:15:25.519182880 +0200
|
|
@@ -54,12 +54,12 @@
|
|
sudo /etc/init.d/virtualbox-guest-utils stop
|
|
sudo /usr/sbin/VBoxService --timesync-set-on-restore --timesync-interval 5000 --timesync-set-threshold 1
|
|
|
|
- # Create /tmp/pgbackrest and give ownership to root so we know unit tests are not writing there
|
|
+ # Create /run/pgbackrest and give ownership to root so we know unit tests are not writing there
|
|
#-----------------------------------------------------------------------------------------------------------------------
|
|
- echo 'Create /tmp/pgbackrest owned by root' && date
|
|
- sudo mkdir -p /tmp/pgbackrest
|
|
- sudo chown root:root /tmp/pgbackrest
|
|
- sudo chmod 700 /tmp/pgbackrest
|
|
+ echo 'Create /run/pgbackrest owned by root' && date
|
|
+ sudo mkdir -p /run/pgbackrest
|
|
+ sudo chown root:root /run/pgbackrest
|
|
+ sudo chmod 700 /run/pgbackrest
|
|
|
|
# Mount tmpfs at /home/vagrant/test for faster testing
|
|
#-----------------------------------------------------------------------------------------------------------------------
|