From a9b3d717eac4c4f3b87b2bb7fe9f0961f4bb6fb0 Mon Sep 17 00:00:00 2001 From: Paul Cristian Sarbu Date: Mon, 29 Jan 2024 18:09:56 +0100 Subject: just-mr: Serve endpoint always implies a remote-execution endpoint As the serve service always has an associated remote-execution endpoint, when just-mr gets passed only a serve endpoint address it will now default to implying a remote-execution endpoint exists too at that address. Additionally, now the implementation of the --backup-to-remote option more clearly shows that it can only be done in native mode. --- src/other_tools/root_maps/distdir_git_map.cpp | 23 ----------------------- 1 file changed, 23 deletions(-) (limited to 'src/other_tools/root_maps/distdir_git_map.cpp') diff --git a/src/other_tools/root_maps/distdir_git_map.cpp b/src/other_tools/root_maps/distdir_git_map.cpp index 2ee0265f..c2ba2e20 100644 --- a/src/other_tools/root_maps/distdir_git_map.cpp +++ b/src/other_tools/root_maps/distdir_git_map.cpp @@ -254,19 +254,6 @@ auto CreateDistdirGitMap( /*fatal=*/true); return; } - // at this point we cannot continue without - // the remote api - if (not remote_api) { - (*logger)( - fmt::format( - "Missing remote-execution " - "endpoint needed to sync " - "workspace root {} with the " - "serve endpoint.", - distdir_tree_id), - /*fatal=*/true); - return; - } // the tree is known locally, so we upload // it to remote CAS for the serve endpoint // to retrieve it and set up the root @@ -401,16 +388,6 @@ auto CreateDistdirGitMap( /*fatal=*/true); return; } - // at this point we cannot continue without the remote api - if (not remote_api) { - (*logger)( - fmt::format("Missing remote-execution endpoint " - "needed to sync workspace root {} with " - "the serve endpoint.", - tree_id), - /*fatal=*/true); - return; - } // try to supply the serve endpoint with the tree via the // remote CAS if (remote_api.value()->IsAvailable({digest})) { -- cgit v1.2.3