Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(335)

Unified Diff: milo/api/proto/buildinfo.proto

Issue 2667353002: milo: Add barebones BuildInfo pRPC service. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: milo/api/proto/buildinfo.proto
diff --git a/milo/api/proto/buildinfo.proto b/milo/api/proto/buildinfo.proto
new file mode 100644
index 0000000000000000000000000000000000000000..df04f73c6052d72fc829592158989afbfaf565e2
--- /dev/null
+++ b/milo/api/proto/buildinfo.proto
@@ -0,0 +1,59 @@
+// Copyright 2017 The LUCI Authors. All rights reserved.
+// Use of this source code is governed under the Apache License, Version 2.0
+// that can be found in the LICENSE file.
+
+syntax = "proto3";
+
+package milo;
+
+import "github.com/luci/luci-go/common/proto/milo/annotations.proto";
+
+// The buildbot service definition.
hinoka 2017/02/02 22:46:23 BuildInfo? Also copypasta the description you sen
dnj 2017/02/03 01:53:26 OK
+service BuildInfo {
+ rpc Get(BuildInfoRequest) returns (BuildInfoResponse);
+}
+
+message BuildInfoRequest {
+ // The request for the name of a BuildBot built.
+ message BuildBot {
+ // The master name.
+ string master_name = 1;
+ // The builder name server.
+ string builder_name = 2;
+ // The build number.
+ int64 build_number = 3;
+ }
+
+ // The request containing a Swarming task.
+ message Swarming {
+ // The Swarming task name.
+ string task = 1;
hinoka 2017/02/02 22:46:23 string server = 1. we have like 8 of them.
dnj 2017/02/03 01:53:26 ATM I'm leaving server off until we have a whiteli
+ }
+
+ oneof build {
+ // Request a BuildBot build.
+ BuildBot buildbot = 1;
+ // Request a Swarming build.
+ Swarming swarming = 2;
+ }
+
+ // Project hint is a LUCI project suggestion for this build. Some builds,
+ // notably older ones, may not contain enough metadata to resolve their
+ // project. Resolution may succeed if this hint is provided and correct.
+ //
+ // This field is optional, and its use is discouraged unless necessary.
+ string project_hint = 11;
hinoka 2017/02/02 22:46:23 Is this actively useful in V1? If not let's remove
dnj 2017/02/03 01:53:26 Yes, it is. Older builds especially, which don't h
+}
+
+// The request containing the name of the master.
+message BuildInfoResponse {
+ // The LUCI project that this build belongs to.
+ string project = 1;
+
+ // The main build step.
+ milo.Step step = 2;
+
+ // The LogDog annotation stream for this build. The Prefix will be populated
+ // and can be used as the prefix for any un-prefixed LogdogStream in "step".
+ milo.LogdogStream annotation_stream = 3;
+}

Powered by Google App Engine
This is Rietveld 408576698