From 1f1324ebed5fabd376895a4a0ecd325062530f4c Mon Sep 17 00:00:00 2001 From: Heikki Linnakangas Date: Tue, 29 Nov 2022 12:54:26 +0200 Subject: [PATCH] Require tenant to be active when calculating tenant size. It's not clear if the calculation would work or make sense, if the tenant is only partially loaded. Let's play it safe, and require it to be Active. --- pageserver/src/http/routes.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pageserver/src/http/routes.rs b/pageserver/src/http/routes.rs index 11d4a202ca..3af4150429 100644 --- a/pageserver/src/http/routes.rs +++ b/pageserver/src/http/routes.rs @@ -577,7 +577,7 @@ async fn tenant_size_handler(request: Request) -> Result, A let tenant_id: TenantId = parse_request_param(&request, "tenant_id")?; check_permission(&request, Some(tenant_id))?; - let tenant = tenant_mgr::get_tenant(tenant_id, false).map_err(ApiError::InternalServerError)?; + let tenant = tenant_mgr::get_tenant(tenant_id, true).map_err(ApiError::InternalServerError)?; // this can be long operation, it currently is not backed by any request coalescing or similar let inputs = tenant