I was planning to make API reflect the current UI, since the UI is well structured already.

The main purpose of API is to stop fuchsia-dashboard.appspot.com from scraping HTML pages :)

I'm taking a stab at this despite REQBY-LUCI-M5-Migration because seems easy && yet educational for me.

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/e472357afe0b83c90c44004977799450e16966b9

commit e472357afe0b83c90c44004977799450e16966b9
Author: tandrii <tandrii@chromium.org>
Date: Tue Jun 20 20:36:12 2017

scheduler: add GetAllJobs api.

BUG= 712427 
R=vadimsh@chromium.org

Review-Url: https://codereview.chromium.org/2945843002

[modify] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/api/scheduler/v1/pb.discovery.go
[modify] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/api/scheduler/v1/scheduler.pb.go
[modify] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/api/scheduler/v1/scheduler.proto
[modify] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/apiservers/scheduler.go
[add] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/apiservers/scheduler_test.go
[add] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/presentation/doc.go
[add] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/presentation/state.go
[add] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/presentation/state_test.go
[modify] https://crrev.com/e472357afe0b83c90c44004977799450e16966b9/scheduler/appengine/ui/presentation.go

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/4ab136ab49c856a5de394d15946eca3eb22143c0

commit 4ab136ab49c856a5de394d15946eca3eb22143c0
Author: tandrii <tandrii@chromium.org>
Date: Wed Jun 21 17:02:14 2017

scheduler: rename JobID to JobName to avoid overloading JobID meaning.

This also renames recently landed GetJobs api Job.Id to Job.Name, and
hence is backwards incompatible, but it is OK API is still experimental.

R=vadimsh@chromium.org
Bug:  712427 
Review-Url: https://codereview.chromium.org/2948103002

[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/api/scheduler/v1/pb.discovery.go
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/api/scheduler/v1/scheduler.pb.go
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/api/scheduler/v1/scheduler.proto
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/apiservers/scheduler.go
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/apiservers/scheduler_test.go
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/engine/engine.go
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/frontend/templates/includes/base.html
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/frontend/templates/pages/invocation.html
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/frontend/templates/pages/job.html
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/frontend/templates/pages/run_job_result.html
[modify] https://crrev.com/4ab136ab49c856a5de394d15946eca3eb22143c0/scheduler/appengine/ui/presentation.go

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/7c3b1e060dd19bad03f926f6cacc220a3182babd

commit 7c3b1e060dd19bad03f926f6cacc220a3182babd
Author: tandrii <tandrii@chromium.org>
Date: Thu Jun 22 17:55:08 2017

scheduler: add GetJobInvocations api.

BUG= 712427 

Review-Url: https://codereview.chromium.org/2948163002

[modify] https://crrev.com/7c3b1e060dd19bad03f926f6cacc220a3182babd/scheduler/api/scheduler/v1/pb.discovery.go
[modify] https://crrev.com/7c3b1e060dd19bad03f926f6cacc220a3182babd/scheduler/api/scheduler/v1/scheduler.pb.go
[modify] https://crrev.com/7c3b1e060dd19bad03f926f6cacc220a3182babd/scheduler/api/scheduler/v1/scheduler.proto
[modify] https://crrev.com/7c3b1e060dd19bad03f926f6cacc220a3182babd/scheduler/appengine/apiservers/scheduler.go
[modify] https://crrev.com/7c3b1e060dd19bad03f926f6cacc220a3182babd/scheduler/appengine/apiservers/scheduler_test.go

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/b080ae464250e8d831a5d8ce09d00d781ae1528b

commit b080ae464250e8d831a5d8ce09d00d781ae1528b
Author: tandrii <tandrii@chromium.org>
Date: Thu Jun 22 18:01:57 2017

scheduler: more of s/JobID/JobName/ where JobID != Job.JobID.

Also moves acls stub from ui to presentation, as the stub will be shared
with apiserver.

R=vadimsh@chromium.org
Bug:  712427 
Review-Url: https://codereview.chromium.org/2949093003

[rename] https://crrev.com/b080ae464250e8d831a5d8ce09d00d781ae1528b/scheduler/appengine/presentation/acl.go
[modify] https://crrev.com/b080ae464250e8d831a5d8ce09d00d781ae1528b/scheduler/appengine/ui/common.go
[modify] https://crrev.com/b080ae464250e8d831a5d8ce09d00d781ae1528b/scheduler/appengine/ui/invocation.go
[modify] https://crrev.com/b080ae464250e8d831a5d8ce09d00d781ae1528b/scheduler/appengine/ui/job.go

I think the last remaining CL is https://codereview.chromium.org/2946343004/
Of course, ACLs for mutation of data are still remaining, as currently they are just allowIfAdmin().

I filed separate bug for ACLs issue 736770.

I believe, read API might be good enough for Fuchsia's case (who currently scrap data off html page). +phosek@ can you confirm this by looking at [1]? LMK if there is smth you need.

[1] https://chromium.googlesource.com/external/github.com/luci/luci-go/+/master/scheduler/api/scheduler/v1/scheduler.proto#10

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/8315cf5fcf6a6048afd984cec0dc32a4027ff4da

commit 8315cf5fcf6a6048afd984cec0dc32a4027ff4da
Author: tandrii <tandrii@chromium.org>
Date: Wed Jul 05 19:55:35 2017

scheduler: add rpcs for actions on job and invocation.

Also:
  * refactor API messages to use JobRef and InvocationRef consistently.
  * introduce named errors to engine

BUG= 712427 

Review-Url: https://codereview.chromium.org/2946343004

[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/api/scheduler/v1/pb.discovery.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/api/scheduler/v1/scheduler.pb.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/api/scheduler/v1/scheduler.proto
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/appengine/apiservers/scheduler.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/appengine/apiservers/scheduler_test.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/appengine/engine/engine.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/appengine/presentation/acl.go
[modify] https://crrev.com/8315cf5fcf6a6048afd984cec0dc32a4027ff4da/scheduler/appengine/presentation/state.go

Deployed to prod as version 1814-8315cf5 

The following revision refers to this bug:
  https://chromium.googlesource.com/external/github.com/luci/luci-go.git/+/e39d552a742c975af28cc5a4b99dbabce6a6ef1d

commit e39d552a742c975af28cc5a4b99dbabce6a6ef1d
Author: tandrii <tandrii@chromium.org>
Date: Fri Jul 07 09:29:29 2017

scheduler: expose paused property of a Job.

Previously, I expected that whether Job is paused can be detected
from it's "current state" string.

However, I just burned 1 hour trying to understand why after calling
PauseJob job's state was still running. The answer is obvious -> latest
invocation was scheduled before my call PauseJob, and hence I just had
to wait.

I think having explicit paused field is actually a better idea, even
though it has a downside of confusing too if state="running" and
paused=true.

R=vadimsh@chromium.org
Bug:  712427 
Review-Url: https://codereview.chromium.org/2967373004

[modify] https://crrev.com/e39d552a742c975af28cc5a4b99dbabce6a6ef1d/scheduler/api/scheduler/v1/pb.discovery.go
[modify] https://crrev.com/e39d552a742c975af28cc5a4b99dbabce6a6ef1d/scheduler/api/scheduler/v1/scheduler.pb.go
[modify] https://crrev.com/e39d552a742c975af28cc5a4b99dbabce6a6ef1d/scheduler/api/scheduler/v1/scheduler.proto
[modify] https://crrev.com/e39d552a742c975af28cc5a4b99dbabce6a6ef1d/scheduler/appengine/apiservers/scheduler.go
[modify] https://crrev.com/e39d552a742c975af28cc5a4b99dbabce6a6ef1d/scheduler/appengine/apiservers/scheduler_test.go