do not register restore callback iif managed

Author: pchalamet

src/Terrabuild/Core/Build.fs

@@ -234,26 +234,26 @@ let run (options: ConfigOptions.Options) (cache: Cache.ICache) (api: Contracts.I
                     |> List.ofSeq
 
                 let callback() =
-                    if node.Managed then
-                        notification.NodeDownloading node
-                        match cache.TryGetSummary allowRemoteCache cacheEntryId with
-                        | Some summary ->
-                            Log.Debug("{NodeId} restoring '{Project}/{Target}' from {Hash}", node.Id, node.Project, node.Target, node.TargetHash)
-                            match summary.Outputs with
-                            | Some outputs ->
-                                let files = IO.enumerateFiles outputs
-                                IO.copyFiles projectDirectory outputs files |> ignore
-                                api |> Option.iter (fun api -> api.UseArtifact node.ProjectHash node.TargetHash)
-                            | _ -> ()
-                            notification.NodeCompleted node TaskRequest.Restore true
-                        | _ ->
-                            notification.NodeCompleted node TaskRequest.Restore false
-                            raiseBugError $"Unable to download build output for {cacheEntryId} for node {node.Id}"
-                    else
-                        Log.Debug("{NodeId} skipping restore '{Project}/{Target}' from {Hash}", node.Id, node.Project, node.Target, node.TargetHash)
-
-                let restorable = Restorable(callback, dependencies)
-                restorables.TryAdd(node.Id, restorable) |> ignore
+                    notification.NodeDownloading node
+                    match cache.TryGetSummary allowRemoteCache cacheEntryId with
+                    | Some summary ->
+                        Log.Debug("{NodeId} restoring '{Project}/{Target}' from {Hash}", node.Id, node.Project, node.Target, node.TargetHash)
+                        match summary.Outputs with
+                        | Some outputs ->
+                            let files = IO.enumerateFiles outputs
+                            IO.copyFiles projectDirectory outputs files |> ignore
+                            api |> Option.iter (fun api -> api.UseArtifact node.ProjectHash node.TargetHash)
+                        | _ -> ()
+                        notification.NodeCompleted node TaskRequest.Restore true
+                    | _ ->
+                        notification.NodeCompleted node TaskRequest.Restore false
+                        raiseBugError $"Unable to download build output for {cacheEntryId} for node {node.Id}"
+
+                if node.Managed then
+                    let restorable = Restorable(callback, dependencies)
+                    restorables.TryAdd(node.Id, restorable) |> ignore
+                else
+                    Log.Debug("{NodeId} skipping restore '{Project}/{Target}' from {Hash}", node.Id, node.Project, node.Target, node.TargetHash)
                 if summary.IsSuccessful then TaskStatus.Success summary.EndedAt
                 else TaskStatus.Failure (summary.EndedAt, $"Restored node {node.Id} with a build in failure state")
             | _ ->