diff options
author | Alberto Sartori <alberto.sartori@huawei.com> | 2023-03-15 12:51:16 +0100 |
---|---|---|
committer | Alberto Sartori <alberto.sartori@huawei.com> | 2023-03-15 14:36:44 +0100 |
commit | 73c00a4820097588c2dc0bd9a89d34a65de0084b (patch) | |
tree | 636a91ceab2fee08dccd1c4ff8c3d81021ac5792 /src | |
parent | 88c6de41173424390400a8937b8db214975b688a (diff) | |
download | justbuild-73c00a4820097588c2dc0bd9a89d34a65de0084b.tar.gz |
bazel client: remove BazelAcClient::UpdateActionResult...
...both buildbarn and just execute do not allow a direct upload to the
action cache. Moreover, our tool does not support it, anyway, meaning
it was dead code.
Diffstat (limited to 'src')
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_ac_client.cpp | 28 | ||||
-rw-r--r-- | src/buildtool/execution_api/remote/bazel/bazel_ac_client.hpp | 6 |
2 files changed, 0 insertions, 34 deletions
diff --git a/src/buildtool/execution_api/remote/bazel/bazel_ac_client.cpp b/src/buildtool/execution_api/remote/bazel/bazel_ac_client.cpp index 6eac5adf..74542453 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_ac_client.cpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_ac_client.cpp @@ -56,31 +56,3 @@ auto BazelAcClient::GetActionResult( } return response; } - -auto BazelAcClient::UpdateActionResult(std::string const& instance_name, - bazel_re::Digest const& action_digest, - bazel_re::ActionResult const& result, - int priority) noexcept - -> std::optional<bazel_re::ActionResult> { - auto policy = std::make_unique<bazel_re::ResultsCachePolicy>(); - policy->set_priority(priority); - - bazel_re::UpdateActionResultRequest request{}; - request.set_instance_name(instance_name); - request.set_allocated_action_digest( - gsl::owner<bazel_re::Digest*>{new bazel_re::Digest{action_digest}}); - request.set_allocated_action_result(gsl::owner<bazel_re::ActionResult*>{ - new bazel_re::ActionResult{result}}); - request.set_allocated_results_cache_policy(policy.release()); - - grpc::ClientContext context; - bazel_re::ActionResult response; - grpc::Status status = - stub_->UpdateActionResult(&context, request, &response); - - if (not status.ok()) { - LogStatus(&logger_, LogLevel::Debug, status); - return std::nullopt; - } - return response; -} diff --git a/src/buildtool/execution_api/remote/bazel/bazel_ac_client.hpp b/src/buildtool/execution_api/remote/bazel/bazel_ac_client.hpp index fd0e79cb..99c28412 100644 --- a/src/buildtool/execution_api/remote/bazel/bazel_ac_client.hpp +++ b/src/buildtool/execution_api/remote/bazel/bazel_ac_client.hpp @@ -39,12 +39,6 @@ class BazelAcClient { std::vector<std::string> const& inline_output_files) noexcept -> std::optional<bazel_re::ActionResult>; - [[nodiscard]] auto UpdateActionResult(std::string const& instance_name, - bazel_re::Digest const& digest, - bazel_re::ActionResult const& result, - int priority) noexcept - -> std::optional<bazel_re::ActionResult>; - private: std::unique_ptr<bazel_re::ActionCache::Stub> stub_; Logger logger_{"RemoteAcClient"}; |