From 0a57a85fc01884104e21546b0bbea6cbcf2314f1 Mon Sep 17 00:00:00 2001 From: Dmitrii Kovalkov Date: Fri, 13 Jun 2025 14:51:42 +0200 Subject: [PATCH] pageserver: add enable-basebackup-cache feature flag for gradual roll out --- pageserver/src/basebackup_cache.rs | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/pageserver/src/basebackup_cache.rs b/pageserver/src/basebackup_cache.rs index 7dde3e02fe..5f2327b4d6 100644 --- a/pageserver/src/basebackup_cache.rs +++ b/pageserver/src/basebackup_cache.rs @@ -413,6 +413,19 @@ impl BasebackupCache { .tenant_manager .get_attached_tenant_shard(tenant_shard_id)?; + let feature_flag = tenant + .feature_resolver + .evaluate_boolean("enable-basebackup-cache", tenant_shard_id.tenant_id); + + if feature_flag.is_err() { + tracing::info!( + tenant_id = %tenant_shard_id.tenant_id, + "Basebackup cache is disabled for tenant by feature flag, skipping basebackup", + ); + self.prepare_skip_count.inc(); + return Ok(()); + } + let tenant_state = tenant.current_state(); if tenant_state != TenantState::Active { anyhow::bail!(