Add BEP (Build Event Protocol) protos #926
Conversation
There was a problem hiding this comment.
fyi: @adam-singer , @aaronmondal
Reviewable status: 0 of 1 LGTMs obtained, and pending CI: Analyze (javascript-typescript), Analyze (python), Bazel Dev / ubuntu-22.04, Cargo Dev / macos-13, Cargo Dev / ubuntu-22.04, Installation / macos-13, Installation / macos-14, Installation / ubuntu-22.04, Local / ubuntu-22.04, Publish image, Publish nativelink-worker-lre-cc, Remote / large-ubuntu-22.04, asan / ubuntu-22.04, docker-compose-compiles-nativelink (20.04), docker-compose-compiles-nativelink (22.04), integration-tests (20.04), integration-tests (22.04), macos-13, pre-commit-checks, ubuntu-20.04 / stable, ubuntu-22.04, ubuntu-22.04 / stable, vale, windows-2022 / stable (waiting on @blakehatch)
There was a problem hiding this comment.
You will want to add another proto that adds RPCs for listening to granular build events and higher-level lifecycle events (build started, build ended, etc) such as this:
Reviewable status: 0 of 1 LGTMs obtained, and pending CI: Bazel Dev / ubuntu-22.04, Cargo Dev / macos-13, Cargo Dev / ubuntu-22.04, Installation / macos-13, Installation / macos-14, Installation / ubuntu-22.04, Local / ubuntu-22.04, Publish image, Publish nativelink-worker-lre-cc, Remote / large-ubuntu-22.04, asan / ubuntu-22.04, docker-compose-compiles-nativelink (20.04), docker-compose-compiles-nativelink (22.04), integration-tests (20.04), integration-tests (22.04), macos-13, ubuntu-20.04 / stable, ubuntu-22.04, ubuntu-22.04 / stable, windows-2022 / stable
There was a problem hiding this comment.
I accidentally added the wrong BEP... apparently bazel has a slightly different proto definition than google's.
Reviewable status: 0 of 1 LGTMs obtained, and pending CI: Vercel, pre-commit-checks (waiting on @blakehatch)
In prep to support the BEP (Build Event Protocol), we are adding the proto definitions to the repo and related generated code. towards: TraceMachina#925
There was a problem hiding this comment.
Reviewed 23 of 23 files at r2, all commit messages.
Reviewable status: 0 of 1 LGTMs obtained (waiting on @blakehatch)
In prep to support the BEP (Build Event Protocol), we are adding the proto definitions to the repo and related generated code.
towards: #925
This change is