diff --git a/pageserver/src/tenant/timeline/detach_ancestor.rs b/pageserver/src/tenant/timeline/detach_ancestor.rs
index 23419ecc82..b3c078312e 100644
--- a/pageserver/src/tenant/timeline/detach_ancestor.rs
+++ b/pageserver/src/tenant/timeline/detach_ancestor.rs
@@ -1139,8 +1139,7 @@ pub(super) async fn detach_and_reparent(
         if let Some(ancestor) = existing {
             Ancestor::Detached(ancestor, ancestor_lsn)
         } else {
-            let direct_children =
-                reparented_direct_children(detached, tenant).map_err(Error::from)?;
+            let direct_children = reparented_direct_children(detached, tenant)?;
             return Ok(DetachingAndReparenting::AlreadyDone(direct_children));
         }
     } else {