From 9fae0d9562bc31ac0704cc7457dd2e09f3318dcb Mon Sep 17 00:00:00 2001 From: Christian Schwarz Date: Thu, 25 Jan 2024 15:48:54 +0000 Subject: [PATCH] using /v1/tenant endpoint is actually not necessary, we don't care about status 503 --- control_plane/src/pageserver.rs | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/control_plane/src/pageserver.rs b/control_plane/src/pageserver.rs index 45ed8fe8b5..783eb8c0b9 100644 --- a/control_plane/src/pageserver.rs +++ b/control_plane/src/pageserver.rs @@ -237,14 +237,13 @@ impl PageServerNode { background_process::InitialPidFile::Expect(self.pid_file()), || async { let res = - tokio::time::timeout(Duration::from_secs(1), self.http_client.list_tenants()) - .await; + tokio::time::timeout(Duration::from_secs(1), self.http_client.status()).await; match res { Ok(res) => match res { Ok(_) => Ok(true), Err(e) => match e { mgmt_api::Error::ReceiveBody(e) => { - if e.is_connect() || e.is_timeout() { + if e.is_connect() { Ok(false) } else { Ok(true)