Add bots.count endpoint

appengine/swarming/handlers_endpoints.py

 # Copyright 2015 The LUCI Authors. All rights reserved.
 # Use of this source code is governed under the Apache License, Version 2.0
 # that can be found in the LICENSE file.
 
 """This module defines Swarming Server endpoints handlers."""
 
 import datetime
 import logging
 
 from google.appengine.api import datastore_errors
@@ skipping 617 matching lines @@
       if len(parts) != 2 or any(i.strip() != i or not i for i in parts):
         raise endpoints.BadRequestException('Invalid dimensions')
       q = q.filter(bot_management.BotInfo.dimensions_flat == d)
     bots, cursor = datastore_utils.fetch_page(q, request.limit, request.cursor)
     return swarming_rpcs.BotList(
         cursor=cursor,
         death_timeout=config.settings().bot_death_timeout_secs,
         items=[message_conversion.bot_info_to_rpc(bot, now) for bot in bots],
         now=now)
 
+  @gae_ts_mon.instrument_endpoint()
+  @auth.endpoints_method(
+      swarming_rpcs.BotsRequest, swarming_rpcs.BotsCount,
+      http_method='GET')
+  @auth.require(acl.is_privileged_user)
+  def count(self, request):
+    """Counts number of bots with given dimensions."""
+    logging.info('%s', request)
+    now = utils.utcnow()
+    q = bot_management.BotInfo.query().order()
+    for d in request.dimensions:
+      parts = d.split(':', 1)
+      if len(parts) != 2 or any(i.strip() != i or not i for i in parts):
+        raise endpoints.BadRequestException('Invalid dimensions: %s' % d)
+      q = q.filter(bot_management.BotInfo.dimensions_flat == d)
+    q_count = q.count_async()
+    dt = datetime.timedelta(seconds=config.settings().bot_death_timeout_secs)
+    timeout = now - dt
+    q_dead = (q.filter(bot_management.BotInfo.last_seen_ts < timeout)

[M-A Ruel, 2016/08/01 19:18:37]

optional style nit: I'd prefer:
q_dead = q.filter(
    bot_management.BotInfo.last_seen_ts < timeout).count_async()

same for the next one.

technically, they are not queries but ndb.Future, so using f_ instead of q_
would make more sense.

[kjlubick, 2016/08/01 19:25:43]

Done.

+        .count_async())
+    q_quarantined = (q.filter(bot_management.BotInfo.quarantined == True)
+        .count_async())
+    q_busy = q.filter(bot_management.BotInfo.is_busy == True).count_async()
+    return swarming_rpcs.BotsCount(
+        count=q_count.get_result(),
+        quarantined=q_quarantined.get_result(),
+        dead=q_dead.get_result(),
+        busy=q_busy.get_result(),
+        now=now)
+
 
 def get_routes():
   return (
     endpoints_webapp2.api_routes(SwarmingServerService) +
     endpoints_webapp2.api_routes(SwarmingTaskService) +
     endpoints_webapp2.api_routes(SwarmingTasksService) +
     endpoints_webapp2.api_routes(SwarmingBotService) +
     endpoints_webapp2.api_routes(SwarmingBotsService) +
     # components.config endpoints for validation and configuring of luci-config
     # service URL.
     endpoints_webapp2.api_routes(config.ConfigApi))