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

Unified Diff: appengine/swarming/handlers_endpoints.py

Issue 2404883002: Add links to MP and resolve minor fixes (Closed)
Patch Set: Address Polymer comments Created 4 years, 2 months 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
Index: appengine/swarming/handlers_endpoints.py
diff --git a/appengine/swarming/handlers_endpoints.py b/appengine/swarming/handlers_endpoints.py
index 03302a2f6f571d89a570ef1763687c58b093fa94..3d95a8d803a62e97715675f909fa649e7c6c1ddc 100644
--- a/appengine/swarming/handlers_endpoints.py
+++ b/appengine/swarming/handlers_endpoints.py
@@ -20,6 +20,7 @@ from protorpc import remote
from components import auth
from components import datastore_utils
from components import endpoints_webapp2
+from components import machine_provider
from components import utils
import message_conversion
@@ -135,9 +136,22 @@ class SwarmingServerService(remote.Service):
def details(self, _request):
"""Returns information about the server."""
host = 'https://' + os.environ['HTTP_HOST']
+
+ mpp = ''
+ if config.settings().mp and config.settings().mp.server:
+ mpp = config.settings().mp.server
+ # as a fallback, try pulling from datastore
+ if not mpp:
+ mpp = machine_provider.MachineProviderConfiguration.get_instance_url()
+ if mpp:
+ mpp = mpp + '/leases/%s'
+
return swarming_rpcs.ServerDetails(
bot_version = bot_code.get_bot_version(host),
- server_version = utils.get_app_version())
+ server_version = utils.get_app_version(),
+ machine_provider_template = mpp,
+ display_server_url_template =
+ config.settings().display_server_url_template)
@gae_ts_mon.instrument_endpoint()
@auth.endpoints_method(
« no previous file with comments | « appengine/swarming/elements/res/imp/taskpage/task-page-demo.html ('k') | appengine/swarming/handlers_endpoints_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698