Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

plugins: postgresql fix missing pg_backup_stop() call #1664

Conversation

bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Jan 15, 2024

Backport of PR #1655 to bareos-23

Checklist for the reviewer of the PR (will be processed by the Bareos team)

Make sure you check/merge the PR using devtools/pr-tool to have some simple automated checks run and a proper changelog record added.

Source code quality (if there were changes to the original PR)
  • Source code changes are understandable
  • Variable and function names are meaningful
  • Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • Original PR plugins: postgresql fix missing pg_backup_stop() call #1655 is merged
  • All functional differences to the original PR are documented above

Copy link
Member

@pstorz pstorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good!

@bruno-at-bareos bruno-at-bareos force-pushed the backport/bareos-23/plugin-postgresql-fix-incrementals branch from fce655d to e10bb90 Compare January 16, 2024 12:56
bruno-at-bareos and others added 2 commits January 16, 2024 13:53
- pg_backup_stop() is called in all job level depending of the
  `backup_running` status. This allow a better support of
  incremental with no changes.
- add in testrunner-default checks of WARNING or ERROR in cluster
  logs
- is_backup_running boolean true if pg_backup_start has been run
- is_backup_full boolean true if backup level is Full
- apply black formatting suggestion

Fixes bareos#1584: Incremental PostgreSQL backup will log an warning on
             the database server

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
(cherry picked from commit b5f2eb1)
@BareosBot BareosBot force-pushed the backport/bareos-23/plugin-postgresql-fix-incrementals branch from 6d89219 to 0f3a5c8 Compare January 16, 2024 13:53
@BareosBot BareosBot merged commit 8e89bfe into bareos:bareos-23 Jan 16, 2024
@bruno-at-bareos bruno-at-bareos deleted the backport/bareos-23/plugin-postgresql-fix-incrementals branch March 13, 2024 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants