[monorail go api] add IssuesList rpc

go/src/infra/monorail/monorail.proto

Index: go/src/infra/monorail/monorail.proto
diff --git a/go/src/infra/monorail/monorail.proto b/go/src/infra/monorail/monorail.proto
index ecec53a0a63540b12cf719305a45178ac34d929a..a9ce937c9918ee5d82186730dd34235c943fed77 100644
--- a/go/src/infra/monorail/monorail.proto
+++ b/go/src/infra/monorail/monorail.proto
@@ -18,6 +18,8 @@ service Monorail {
     rpc InsertIssue(InsertIssueRequest) returns (InsertIssueResponse){};
     // Posts a comment to an issue. Can update issue attributes, such as status.
     rpc InsertComment(InsertCommentRequest) returns (InsertCommentResponse){};
+    // Lists issues from a project.
+    rpc IssuesList(IssuesListRequest) returns (IssuesListResponse){};
 }
 
 // A monorail issue.
@@ -100,6 +102,39 @@ message InsertCommentRequest {
 
 message InsertCommentResponse{}
 
+message IssuesListRequest {

[nodir, 2016/06/09 04:09:28]

Comments world be great, otherwise a reader may have hard time understanding
what these fields mean. Other fields and messages in this file are documented

[seanmccullough1, 2016/06/09 15:01:33]

Done.

+  string projectId = 1;
+  repeated string additionalProject = 2;
+  enum CannedQuery {
+    ALL = 0;
+    NEW = 1;
+    OPEN = 2;
+    OWNED = 3;
+    REPORTED = 4;
+    STARRED = 5;
+    TO_VERIFY = 6;
+  }
+  CannedQuery can= 3;

[martiniss, 2016/06/09 02:30:13]

space around =

[seanmccullough1, 2016/06/09 15:01:33]

Done.

+  string label = 4;
+  int32 maxResults = 5;
+  string owner = 6;
+  int64 publishedMax = 7;
+  int64 publishedMin = 8;
+  string q = 9;
+  string sort = 10;
+  int32 startIndex = 11;
+  string status = 12;
+  int64 updatedMax = 13;
+  int64 updatedMin = 14;
+}
+
+message IssuesListResponse {
+  // error = messages.MessageField(ErrorMessage, 1)

[nodir, 2016/06/09 04:09:28]

Looks like an important field. Why not included?

[seanmccullough1, 2016/06/09 15:01:33]

It was ignored by other Request protos defined here. Added it for this one.

+  repeated Issue items = 2;
+  int32 totalResults = 3;
+  string kind = 4;

[martiniss, 2016/06/09 02:30:13]

what is this exactly?

[seanmccullough1, 2016/06/09 15:01:32]

I could defer to the original doc:
https://chromium.googlesource.com/infra/infra/+/f2e2a4d/appengine/monorail/pr...
:)

I believe it is an endpoints thing, for this response I assume it will always be
"monorail#issueList"

+}
+
 // Defines a mutation to an issue.
 // This message is partial.
 // Derived from Update type in api_pb2_v1.py.