we have two options: switch to Cloud Endpoints v2 and use webapp2 adapter.
the more we thinkg about this problem, the more dependencies on the discovery service we find. The webapp2 adapter does not support discovery service. It didn't support it from the start because we were piggy-backing on endpoints's discovery doc. The workflow was and still is:
1) in python, build a client dynamically based on the discovery doc returned by endpoints v1
2) modify client's base URL to remove _ah to use webapp2-adapter's URLs
3) use the client
this workflow will stop working one endpoints v1 is down.
This won't affect buildbot-buildbucket integration though. I've committed buildbucket's discovery doc in the repo a while ago and buildbot masters load it from the file system.
The following revision refers to this bug:
https://chromium.googlesource.com/infra/infra/+/f0fd4d4c4f642ee062580933af2c3b9a21ef452e
commit f0fd4d4c4f642ee062580933af2c3b9a21ef452e
Author: smut <smut@google.com>
Date: Wed Jun 06 23:14:53 2018
Roll luci-go to 2d92ca945b664b7cc438eef33849dcee48484cea
Changes:
2d92ca94 [Swarming] Update client to hit /_ah/api instead of /api
5ffb5ff1 Update Buildbucket clients to hit /_ah/api instead of /api
b93923db stringset: switch Set to a concrete type
16aa1a54 [logdog] Remove dispatchThreshold
9883c80d [appengine] Make GetRPCTransport respect per-request context and deadlines.
318be9c1 [milo] stop warning that undefined faviconURL is not valid.
2933a617 [cipd] Never overwrite published files in GS.
7d10327c [milo] Fix search page QoL bugs
521b5fb3 [milo] Remove "fix datastore" cron job.
a2686393 [luci-notify] require luci-notify.cfg
e8e20691 archiver: unexport ItemBundle and ShardItems
1bd1967d archiver: move tar archiver from cmd/isolate
b0580dda archiver: rename Item to PendingItem
7057582a [luci-notify] Log build id in pubsub handler
a2bb9574 [rpcexplorer] Add support for field masks
6db112c5 [scheduler] Move config protos to 'scheduler.config' proto package.
a9d76f7b Add myself to OWNERS
c0a1daed milo: detect and show recipe when recipe bundle is used.
5fbd65f9 Fix luci-go after updating dependencies.
753b186a isolate: Remove ArchiveOptions reference from TarringArchiver
32e861db Remove WATCHLISTS as it is no longer used by anyone.
Bug: 805543
Change-Id: I3f45c1ca3637db641cdee574991cf3f6d3cbde89
Reviewed-on: https://chromium-review.googlesource.com/1089889
Commit-Queue: smut <smut@google.com>
Reviewed-by: Andrii Shyshkalov <tandrii@chromium.org>
[modify] https://crrev.com/f0fd4d4c4f642ee062580933af2c3b9a21ef452e/DEPS
[modify] https://crrev.com/f0fd4d4c4f642ee062580933af2c3b9a21ef452e/go/src/infra/tools/led/isolate.go
Comment 1 by no...@chromium.org
, Jan 24 2018