pageserver: basebackup cache follow-up 1 #11998

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.

Sign up for GitHub

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

Jump to bottom

Closed

DimasKovas wants to merge 1 commit into main from diko/basebackup_cache_followup_1

+3 −1

Member

DimasKovas commented May 22, 2025

Problem

Follow up on comment from previous PR pageserver: basebackup cache (hackathon project) #11989 (comment)

Summary of changes

Make it more clear that we don't do anything in walingest if the feature flag is off


          follow up comments

892dcde

DimasKovas requested a review from VladLazar

May 22, 2025 13:19

DimasKovas requested a review from a team as a code owner

May 22, 2025 13:19

github-actions bot commented May 22, 2025

8492 tests run: 7944 passed, 0 failed, 548 skipped (full report)

Flaky tests (4)

Postgres 15

test_timeline_copy[100000]: release-x86-64-with-lfc

Postgres 14

test_lfc_prewarm_under_workload[compute-ctl]: release-x86-64-with-lfc
test_cancellations: release-arm64-with-lfc
test_timeline_copy[100000]: release-x86-64-with-lfc

Code coverage* (full report)

functions: 32.4% (9050 of 27919 functions)
lines: 48.7% (79412 of 163157 lines)

* collected from Rust tests only

_{The comment gets automatically updated with the latest test results
892dcde at 2025-05-22T14:19:31.914Z :recycle:}

DimasKovas enabled auto-merge

May 26, 2025 07:59

DimasKovas closed this

auto-merge was automatically disabled

July 31, 2025 12:41

Pull request was closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet