Skip to content

Commit 3d730eb

Browse files
authored
Logging: reduce logs in task router (#8854)
This is well tested already and we haven't find any problem with it. I'm leaving only the INFO logs when we are changing the routing.
1 parent ca6a82c commit 3d730eb

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

readthedocs/builds/tasks.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,17 +66,17 @@ class TaskRouter:
6666
BUILD_LARGE_QUEUE = 'build:large'
6767

6868
def route_for_task(self, task, args, kwargs, **__):
69-
log.info('Executing TaskRouter.', task=task)
69+
log.debug('Executing TaskRouter.', task=task)
7070
if task not in (
7171
'readthedocs.projects.tasks.update_docs_task',
7272
'readthedocs.projects.tasks.sync_repository_task',
7373
):
74-
log.info('Skipping routing non-build task.', task=task)
74+
log.debug('Skipping routing non-build task.', task=task)
7575
return
7676

7777
version = self._get_version(task, args, kwargs)
7878
if not version:
79-
log.info('No Build/Version found. No routing task.', task=task)
79+
log.debug('No Build/Version found. No routing task.', task=task)
8080
return
8181

8282
project = version.project
@@ -145,7 +145,7 @@ def route_for_task(self, task, args, kwargs, **__):
145145
)
146146
return self.BUILD_LARGE_QUEUE
147147

148-
log.info(
148+
log.debug(
149149
'No routing task because no conditions were met.',
150150
project_slug=project.slug,
151151
)
@@ -162,7 +162,7 @@ def _get_version(self, task, args, kwargs):
162162
try:
163163
version = Version.objects.get(pk=version_pk)
164164
except Version.DoesNotExist:
165-
log.info(
165+
log.debug(
166166
'Version does not exist. Routing task to default queue.',
167167
version_id=version_pk,
168168
)

0 commit comments

Comments
 (0)