Skip to content

Add empty_db for pg_dump directory checks#65

Merged
Jetman80 merged 3 commits intomasterfrom
empty_db-pgdump-directory-checks
Mar 14, 2026
Merged

Add empty_db for pg_dump directory checks#65
Jetman80 merged 3 commits intomasterfrom
empty_db-pgdump-directory-checks

Conversation

@Raven-98
Copy link
Copy Markdown
Contributor

@Raven-98 Raven-98 commented Mar 13, 2026

This pull request updates the backup verification logic to better handle "empty" databases by skipping certain checks for them. Specifically, it adds logic to bypass the toc.dat file size and TABLE DATA entries checks for databases marked as empty, reducing unnecessary error logs for expected-empty databases.

Improvements to backup verification for empty databases:

  • Skips the toc.dat file size check for databases listed in the empty_db configuration, logging a notice instead of an error when this is the case.
  • Skips the TABLE DATA entries count check for databases listed in the empty_db configuration, logging a notice instead of an error when this is the case.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds support for empty_db to the PostgreSQL pg_dump directory backup verification path so that known-empty databases don’t fail size/content checks during --check runs.

Changes:

  • Skip toc.dat minimum size validation when the database is listed in check.empty_db (or ALL).
  • Skip TABLE DATA entry count validation when the database is listed in check.empty_db (or ALL).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread rsnapshot_backup/rsnapshot_backup.py Outdated
Comment thread rsnapshot_backup/rsnapshot_backup.py Outdated
Jetman80 and others added 2 commits March 14, 2026 20:20
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
Co-authored-by: Copilot Autofix powered by AI <[email protected]>
@Jetman80 Jetman80 merged commit 693a2e5 into master Mar 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants