Commit 7c5cf152 authored by rockot's avatar rockot Committed by Commit bot

Docserver: Fix API refresh task names

So these tasks have been queueing with different path names than
those used by the template processor. The latter uses "Unix name"
equivalents of the API features, while the former was not.

For example, "apps/appviewTag" has been caching as such but the
template looks for "apps/appview_tag". It's unclear when this
incongruity appeared but it explains several "my-docs-arent-updating"
bugs that have appeared in the past several weeks.

This changes the refresh servlet to queue API refresh tasks using
API features in unix_name form, resolving the incongruity.

BUG=471019
TBR=kalman@chromium.org

Review URL: https://codereview.chromium.org/1038993002

Cr-Commit-Position: refs/heads/master@{#322496}
parent 0c169fe7
......@@ -11,6 +11,7 @@ from file_system import FileNotFoundError
from future import Future, All
from jsc_view import CreateJSCView, GetEventByNameFromEvents
from platform_util import GetPlatforms
from third_party.json_schema_compiler.model import UnixName
class APIDataSource(DataSource):
......@@ -87,7 +88,7 @@ class APIDataSource(DataSource):
def GetRefreshPaths(self):
tasks = []
for platform in GetPlatforms():
tasks += ['%s/%s' % (platform, api)
tasks += ['%s/%s' % (platform, UnixName(api))
for api in
self._platform_bundle.GetAPIModels(platform).GetNames()]
return tasks
......
application: chrome-apps-doc
version: 3-48-0
version: 3-48-2
runtime: python27
api_version: 1
threadsafe: false
......
......@@ -2,4 +2,4 @@ cron:
- description: Repopulates all cached data.
url: /_cron
schedule: every 180 minutes
target: 3-48-0
target: 3-48-2
queue:
- name: default
rate: 7/s
bucket_size: 7
max_concurrent_requests: 7
rate: 4/s
bucket_size: 4
max_concurrent_requests: 4
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment