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

Unified Diff: appengine/swarming/handlers_backend.py

Issue 2249803003: Create Tasks.Tags endpoint and Cron job (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-py@master
Patch Set: Docs Created 4 years, 4 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
« no previous file with comments | « appengine/swarming/cron.yaml ('k') | appengine/swarming/handlers_endpoints.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/swarming/handlers_backend.py
diff --git a/appengine/swarming/handlers_backend.py b/appengine/swarming/handlers_backend.py
index ee398790eeacbe1e2a39a0ee8a833567ce537c23..82659a01fc8d2328bac0f8f22dc380da32de28e0 100644
--- a/appengine/swarming/handlers_backend.py
+++ b/appengine/swarming/handlers_backend.py
@@ -4,6 +4,7 @@
"""Main entry point for Swarming backend handlers."""
+import datetime
import json
import logging
@@ -16,11 +17,13 @@ from components import utils
import mapreduce_jobs
from components import decorators
+from components import datastore_utils
from components import machine_provider
from server import bot_management
from server import config
from server import lease_management
from server import stats
+from server import task_result
from server import task_scheduler
@@ -124,6 +127,40 @@ class CronBotsDimensionAggregationHandler(webapp2.RequestHandler):
ts=now).put()
+class CronTasksTagsAggregationHandler(webapp2.RequestHandler):
+ """Aggregates all task tags from the last 12 hours."""
+
+ @decorators.require_cronjob
+ def get(self):
+ seen = {}
+ now = utils.utcnow()
+ count = 0
+ q = task_result.get_result_summaries_query(
+ now - datetime.timedelta(hours=12), None, 'created_ts', 'all', None)
+ cursor = None
+ while True:
+ tasks, cursor = datastore_utils.fetch_page(q, 1000, cursor)
+ count += len(tasks)
+ for t in tasks:
+ for i in t.tags:
+ k, v = i.split(':', 1)
+ if k != 'cron_invocation_id':
+ seen.setdefault(k, set()).add(v)
+ if not cursor or len(tasks) == 0:
+ break
+
+ tags = [
+ task_result.TagValues(tag=k, values=sorted(values))
+ for k, values in sorted(seen.iteritems())
+ ]
+
+ logging.info('From %d tasks, saw tags %s', count, tags)
+ task_result.TagAggregation(
+ key=task_result.TagAggregation.KEY,
+ tags=tags,
+ ts=now).put()
+
+
class CronMachineProviderPubSubHandler(webapp2.RequestHandler):
"""Listens for Pub/Sub communication from Machine Provider."""
@@ -199,6 +236,8 @@ def get_routes():
('/internal/cron/trigger_cleanup_data', CronTriggerCleanupDataHandler),
('/internal/cron/aggregate_bots_dimensions',
CronBotsDimensionAggregationHandler),
+ ('/internal/cron/aggregate_tasks_tags',
+ CronTasksTagsAggregationHandler),
('/internal/cron/machine_provider', CronMachineProviderBotHandler),
('/internal/cron/machine_provider_cleanup',
CronMachineProviderCleanUpHandler),
« no previous file with comments | « appengine/swarming/cron.yaml ('k') | appengine/swarming/handlers_endpoints.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698