LogDog: Enable Coordinator to load project configs

LogDog: Enable Coordinator to load project configs

Add facilities to Coordinator service to load project and service
configurations.

Endpoints will not use any new configurations yet.

BUG=

Committed: https://github.com/luci/luci-go/commit/6dbddb374dacb9f054c38c9164df04cc7b17cd98

[dnj (Google), 2016-05-11 01:26:55]

dnj@google.com changed reviewers:
+ dnj@google.com, hinoka@chromium.org, vadimsh@chromium.org

[dnj (Google), 2016-05-11 01:26:55]

PTAL. Nothing actually *uses* this yet, but small CLs :)

[dnj, 2016-05-17 20:30:19]

Description was changed from

==========
LogDog: Enable Coordinator to load project configs

BUG=
==========

to

==========
LogDog: Enable Coordinator to load project configs

Add facilities to Coordinator service to load project and service
configurations.

Endpoints will not use any new configurations yet.

BUG=
==========

[nodir, 2016-05-18 16:05:34]

nodir@chromium.org changed reviewers:
+ nodir@chromium.org

[nodir, 2016-05-18 16:05:35]

lgtm

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
File appengine/logdog/coordinator/config/projects.go (right):

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/config/projects.go:34: // ProjectConfig loads the
the project config protobuf from the config service.
the the

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/config/projects.go:50: "configSet":  
cfg.ConfigSet,
somewhat sort keys?

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
File appengine/logdog/coordinator/coordinatorTest/service.go (right):

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/coordinatorTest/service.go:24: // PC, if nto nil,
will be used to get the return values for ProjectConfig,
typo in not

[dnj (Google), 2016-05-18 16:46:07]

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
File appengine/logdog/coordinator/config/projects.go (right):

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/config/projects.go:34: // ProjectConfig loads the
the project config protobuf from the config service.
On 2016/05/18 16:05:35, nodir wrote:
> the the

Done.

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/config/projects.go:50: "configSet":  
cfg.ConfigSet,
On 2016/05/18 16:05:35, nodir wrote:
> somewhat sort keys?

I'll put them in a logically-useful order, so in this case more specialized as
it goes down.

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
File appengine/logdog/coordinator/coordinatorTest/service.go (right):

https://codereview.chromium.org/1971623002/diff/1/appengine/logdog/coordinato...
appengine/logdog/coordinator/coordinatorTest/service.go:24: // PC, if nto nil,
will be used to get the return values for ProjectConfig,
On 2016/05/18 16:05:35, nodir wrote:
> typo in not

Done.

[dnj (Google), 2016-05-19 23:02:48]

The CQ bit was checked by dnj@google.com

[dnj (Google), 2016-05-19 23:02:48]

The patchset sent to the CQ was uploaded after l-g-t-m from nodir@chromium.org
Link to the patchset: https://codereview.chromium.org/1971623002/#ps80001
(title: "rebase (auto)")

[commit-bot: I haz the power, 2016-05-19 23:02:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1971623002/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1971623002/80001

[commit-bot: I haz the power, 2016-05-19 23:06:27]

Description was changed from

==========
LogDog: Enable Coordinator to load project configs

Add facilities to Coordinator service to load project and service
configurations.

Endpoints will not use any new configurations yet.

BUG=
==========

to

==========
LogDog: Enable Coordinator to load project configs

Add facilities to Coordinator service to load project and service
configurations.

Endpoints will not use any new configurations yet.

BUG=

Committed:
https://github.com/luci/luci-go/commit/6dbddb374dacb9f054c38c9164df04cc7b17cd98
==========

[commit-bot: I haz the power, 2016-05-19 23:06:28]

Committed patchset #5 (id:80001) as
https://github.com/luci/luci-go/commit/6dbddb374dacb9f054c38c9164df04cc7b17cd98