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

Unified Diff: logdog/appengine/coordinator/service.go

Issue 2591443002: Use classic GAE for coordinator default frontend. (Closed)
Patch Set: Rebase Created 4 years 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
« no previous file with comments | « logdog/appengine/cmd/coordinator/vmuser/module.cfg ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: logdog/appengine/coordinator/service.go
diff --git a/logdog/appengine/coordinator/service.go b/logdog/appengine/coordinator/service.go
index 4a451e654eab64bb4be044ef7c373558988736f8..d4b43b741aa869fc3497060cb589c62e210b40b8 100644
--- a/logdog/appengine/coordinator/service.go
+++ b/logdog/appengine/coordinator/service.go
@@ -257,12 +257,13 @@ func (s *prodServicesInst) newBigTableStorage(c context.Context) (Storage, error
return nil, errors.New("failed to create BigTable credentials")
}
+ // Use an AppEngine Context so we have access to the socket API. This is
+ // needed by AppEngine Classic for gRPC connections.
+ //
// Explicitly clear gRPC metadata from the Context. It is forwarded to
// delegate calls by default, and standard request metadata can break BigTable
// calls.
- c = metadata.NewContext(c, nil)
-
- st, err := bigtable.New(c, bigtable.Options{
+ st, err := bigtable.New(metadata.NewContext(s.aeCtx, nil), bigtable.Options{
Project: bt.Project,
Instance: bt.Instance,
LogTable: bt.LogTableName,
« no previous file with comments | « logdog/appengine/cmd/coordinator/vmuser/module.cfg ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698