Coordinator: AppEngine context for Pub/Sub.

Coordinator: AppEngine context for Pub/Sub.

After switching Pub/Sub to gRPC, we need to pass the AppEngine Context
instead of the luci/gae Context.

BUG=chromium:675401
TEST=None
R=vadimsh@chromium.org

Review-Url: https://codereview.chromium.org/2584293002
Committed: https://github.com/luci/luci-go/commit/0da66c1a83109bf1973a7646efec7331bb8f5d41

[dnj, 2016-12-18 21:38:04]

Description was changed from

==========
Coordinator: AppEngine context for Pub/Sub.

After switching Pub/Sub to gRPC, we need to pass the AppEngine Context
instead of the luci/gae Context.

BUG=None
TEST=None
R=vadimsh@chromium.org
==========

to

==========
Coordinator: AppEngine context for Pub/Sub.

After switching Pub/Sub to gRPC, we need to pass the AppEngine Context
instead of the luci/gae Context.

BUG=chromium:675401
TEST=None
R=vadimsh@chromium.org
==========

[Vadim Sh., 2016-12-18 21:40:10]

lgtm with nit

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

https://codereview.chromium.org/2584293002/diff/1/logdog/appengine/coordinato...
logdog/appengine/coordinator/service.go:346: // Don't pass gRPC metadata to
PubSub.
nit: move this comment to the list 343.

[dnj, 2016-12-18 21:41:08]

The CQ bit was checked by dnj@chromium.org

[dnj, 2016-12-18 21:41:08]

The patchset sent to the CQ was uploaded after l-g-t-m from vadimsh@chromium.org
Link to the patchset: https://codereview.chromium.org/2584293002/#ps20001
(title: "Comment")

[commit-bot: I haz the power, 2016-12-18 21:41:18]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-18 21:46:54]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1482097268349770,
"parent_rev": "98994fd39f063c3a7f6292e57fcdc575c8d71d4d", "commit_rev":
"0da66c1a83109bf1973a7646efec7331bb8f5d41"}

[commit-bot: I haz the power, 2016-12-18 21:46:56]

Description was changed from

==========
Coordinator: AppEngine context for Pub/Sub.

After switching Pub/Sub to gRPC, we need to pass the AppEngine Context
instead of the luci/gae Context.

BUG=chromium:675401
TEST=None
R=vadimsh@chromium.org
==========

to

==========
Coordinator: AppEngine context for Pub/Sub.

After switching Pub/Sub to gRPC, we need to pass the AppEngine Context
instead of the luci/gae Context.

BUG=chromium:675401
TEST=None
R=vadimsh@chromium.org

Review-Url: https://codereview.chromium.org/2584293002
Committed:
https://github.com/luci/luci-go/commit/0da66c1a83109bf1973a7646efec7331bb8f5d41
==========

[commit-bot: I haz the power, 2016-12-18 21:46:56]

Committed patchset #2 (id:20001) as
https://github.com/luci/luci-go/commit/0da66c1a83109bf1973a7646efec7331bb8f5d41