From fc4d80bbf25f7c3e5ce702cd647ac3e18586256b Mon Sep 17 00:00:00 2001
From: Joonas Koivunen <joonas@neon.tech>
Date: Wed, 24 Jul 2024 12:37:00 +0000
Subject: [PATCH] elaborate on TODO for which a test is later added

---
 pageserver/src/tenant/timeline/detach_ancestor.rs | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/pageserver/src/tenant/timeline/detach_ancestor.rs b/pageserver/src/tenant/timeline/detach_ancestor.rs
index 56c641ee49..8ed1072169 100644
--- a/pageserver/src/tenant/timeline/detach_ancestor.rs
+++ b/pageserver/src/tenant/timeline/detach_ancestor.rs
@@ -297,6 +297,8 @@ impl SharedState {
         // TODO: pause failpoint here to catch the situation where detached timeline is deleted...?
         // we are not yet holding the gate so it could advance to the point of removing from
         // timelines.
+        //
+        // pause failpoint which triggers after activating but before completing here
 
         let Some(timeline) = tenant
             .timelines