Skip to content

Make pg_rewind work with encrypted WAL #335

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 3 commits into
base: TDE_REL_17_STABLE
Choose a base branch
from

Conversation

dAdAbird
Copy link
Member

@dAdAbird dAdAbird commented May 14, 2025

The main goal was to pass pg_rewind tests to find and fix WAL-related issues. Along with that, now we have pg_rewind working with encrypted WAL. There are no changes in the user interface.

This PR omits one of the pg_rewind tests. That fails due to pg_basebackup (see PG-1603).

For PG-1037, PG-1412

@dAdAbird dAdAbird requested a review from dutow as a code owner May 14, 2025 15:44
@dAdAbird dAdAbird requested a review from AndersAstrand May 14, 2025 15:45
dAdAbird added 3 commits May 14, 2025 19:12
These tests are a copy of upstream's pg_rewind tests with small changes
in setup_cluster to set WAL encryption on. `standby_source` test is not
copied as it fails because of changes needed in pg_basebackup.
We use TLI to calculate encryption IV, therefore, data in segments has
to be re-encrypted when copied to the new timeline.

For PG-1412
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.69%. Comparing base (443d33c) to head (4e2a192).
Report is 1 commits behind head on TDE_REL_17_STABLE.

❌ Your project status has failed because the head coverage (80.69%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files
@@                  Coverage Diff                  @@
##           TDE_REL_17_STABLE     #335      +/-   ##
=====================================================
+ Coverage              80.54%   80.69%   +0.15%     
=====================================================
  Files                     22       22              
  Lines                   2606     2606              
  Branches                 400      399       -1     
=====================================================
+ Hits                    2099     2103       +4     
+ Misses                   428      426       -2     
+ Partials                  79       77       -2     
Components Coverage Δ
access 83.65% <ø> (+0.69%) ⬆️
catalog 87.55% <ø> (ø)
common 91.80% <ø> (ø)
encryption 76.63% <ø> (+0.93%) ⬆️
keyring 72.00% <ø> (-0.31%) ⬇️
src 66.74% <ø> (ø)
smgr 97.29% <ø> (+0.02%) ⬆️
transam ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants