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

Unified Diff: scheduler/api/scheduler/v1/scheduler.proto

Issue 2945843002: scheduler WIP: add GetAllJobs api. (Closed)
Patch Set: Created 3 years, 6 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: scheduler/api/scheduler/v1/scheduler.proto
diff --git a/scheduler/api/scheduler/v1/scheduler.proto b/scheduler/api/scheduler/v1/scheduler.proto
index 3926529a61a9a9fe35b08c14380cd89dabafe263..0aea7be1da382ab5b2980551ed16fa6049066b51 100644
--- a/scheduler/api/scheduler/v1/scheduler.proto
+++ b/scheduler/api/scheduler/v1/scheduler.proto
@@ -9,5 +9,29 @@ package scheduler;
// Scheduler exposes the public API of the Scheduler service.
service Scheduler {
- // TODO
+ // TODO: still in progress.
+
+ // GetJobs fetches all jobs satisfying JobsRequest and visibility ACLs.
+ rpc GetJobs(JobsRequest) returns (JobsReply);
+}
+
+message JobsRequest {
+ // If not specified or "", all projects' jobs are returned.
+ string project = 1;
+}
+
+message JobsReply {
+ repeated JobWithState jobs = 1;
+}
+
+message JobWithState {
Vadim Sh. 2017/06/19 17:44:19 just "Job" if we need it without state, we can se
tandrii(chromium) 2017/06/20 11:48:05 Done.
+ string id = 1;
+ string project = 2;
+ string schedule = 3;
+
+ JobState state = 4;
+}
+
+message JobState {
+ string kind = 1;
Vadim Sh. 2017/06/19 17:44:19 so... there are actually two flavors of state: one
tandrii(chromium) 2017/06/20 11:48:05 I agree that UI one is more useful; however, if in
Vadim Sh. 2017/06/20 17:48:54 Yes, I meant, the API should not expose internal j
}

Powered by Google App Engine
This is Rietveld 408576698