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

Side by Side 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 unified diff | Download patch
OLDNEW
1 // Copyright 2016 The LUCI Authors. All rights reserved. 1 // Copyright 2016 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 syntax = "proto3"; 5 syntax = "proto3";
6 6
7 package scheduler; 7 package scheduler;
8 8
9 9
10 // Scheduler exposes the public API of the Scheduler service. 10 // Scheduler exposes the public API of the Scheduler service.
11 service Scheduler { 11 service Scheduler {
12 // TODO 12 // TODO: still in progress.
13
14 // GetJobs fetches all jobs satisfying JobsRequest and visibility ACLs.
15 rpc GetJobs(JobsRequest) returns (JobsReply);
13 } 16 }
17
18 message JobsRequest {
19 // If not specified or "", all projects' jobs are returned.
20 string project = 1;
21 }
22
23 message JobsReply {
24 repeated JobWithState jobs = 1;
25 }
26
27 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.
28 string id = 1;
29 string project = 2;
30 string schedule = 3;
31
32 JobState state = 4;
33 }
34
35 message JobState {
36 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
37 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698