Skip to content

LogStream: Use `remote_logstream.proto` from bazelbuild/remote-apis

Santiago Gil requested to merge santigl/logstream-proto-v1 into master

Before raising this MR, consider whether the following are required, and complete if so:

  • Unit tests
  • Metrics
  • Documentation update(s)

If not required, please explain in brief why not.

This switches protobufs that are functionally equivalent.

Description

This replaces the LogStream protobuf for the one that is set to be added to bazelbuild/remote-apis.

The specification remains unchanged.

Changes proposed in this merge request:

  • Add new protobuf file
  • Update the imports (the proto is now under build.bazel.remote.logstream.v1 and it's name is remote_logstream.proto)

Validation

Since the actual specification has not changed, this modification does not introduce functional changes.

Issues addressed

This will allow buildbox-common (after BuildGrid/buildbox/buildbox-common!320 is merged) to issue CreateLogStream() requests to BuildGrid. Without the changes in this MR, versions would not match and that would not be possible.

Merge request reports