This change adds the writing of the remote execution log to a file behind an experimental flag. It also adds a logging handler for Execute calls so that they are logged.

PiperOrigin-RevId: 190991493

2 files changed, 22 insertions, 0 deletions

diff --git a/src/main/protobuf/BUILD b/src/main/protobuf/BUILD
index 4154a17fa5..3ed9d1f541 100644
--- a/src/main/protobuf/BUILD
+++ b/src/main/protobuf/BUILD
@@ -133,6 +133,7 @@ proto_library(
     srcs = ["remote_execution_log.proto"],
     deps = [
         "@googleapis//:google_devtools_remoteexecution_v1test_remote_execution_proto",
+        "@googleapis//:google_longrunning_operations_proto",
         "@googleapis//:google_rpc_status_proto",
     ],
 )
diff --git a/src/main/protobuf/remote_execution_log.proto b/src/main/protobuf/remote_execution_log.proto
index c685bd3fff..81c8de9f94 100644
--- a/src/main/protobuf/remote_execution_log.proto
+++ b/src/main/protobuf/remote_execution_log.proto
@@ -17,6 +17,7 @@ syntax = "proto3";
 package remote_logging;
 
 import "google/devtools/remoteexecution/v1test/remote_execution.proto";
+import "google/longrunning/operations.proto";
 import "google/rpc/status.proto";
 
 option java_package = "com.google.devtools.build.lib.remote.logging";
@@ -33,4 +34,24 @@ message LogEntry {
   // io.grpc.MethodDescriptor.getFullMethodName() (i.e. in format
   // $FULL_SERVICE_NAME/$METHOD_NAME).
   string method_name = 3;
+
+  // Method specific details for this call.
+  RpcCallDetails details = 4;
+}
+
+// Details for a call to
+// google.devtools.remoteexecution.v1test.Execution.Execute.
+message ExecuteDetails {
+  // The google.devtools.remoteexecution.v1test.ExecuteRequest sent by the
+  // call.
+  google.devtools.remoteexecution.v1test.ExecuteRequest request = 1;
+  // The google.longrunning.Operation received by the Execute call.
+  google.longrunning.Operation response = 2;
+}
+
+// Contains details for specific types of calls.
+message RpcCallDetails {
+  oneof details {
+    ExecuteDetails execute = 1;
+  }
 }