Logging: reduce log verbosity (#9348)

Author: humitos

readthedocs/builds/tasks.py

@@ -210,12 +210,22 @@ def archive_builds_task(self, days=14, limit=200, delete=False):
             commands = BuildCommandSerializer(build.commands, many=True).data
             if commands:
                 for cmd in commands:
-                    if len(cmd['output']) > MAX_BUILD_COMMAND_SIZE:
-                        cmd['output'] = cmd['output'][-MAX_BUILD_COMMAND_SIZE:]
-                        cmd['output'] = "... (truncated) ...\n\nCommand output too long. Truncated to last 1MB.\n\n" + cmd['output']  # noqa
-                        log.warning('Truncating build command for build.', build_id=build.id)
-                output = BytesIO(json.dumps(commands).encode('utf8'))
-                filename = '{date}/{id}.json'.format(date=str(build.date.date()), id=build.id)
+                    if len(cmd["output"]) > MAX_BUILD_COMMAND_SIZE:
+                        cmd["output"] = cmd["output"][-MAX_BUILD_COMMAND_SIZE:]
+                        cmd["output"] = (
+                            "\n\n"
+                            "... (truncated) ..."
+                            "\n\n"
+                            "Command output too long. Truncated to last 1MB."
+                            "\n\n" + cmd["output"]
+                        )  # noqa
+                        log.debug(
+                            "Truncating build command for build.", build_id=build.id
+                        )
+                output = BytesIO(json.dumps(commands).encode("utf8"))
+                filename = "{date}/{id}.json".format(
+                    date=str(build.date.date()), id=build.id
+                )
                 try:
                     build_commands_storage.save(name=filename, content=output)
                     if delete:

readthedocs/proxito/middleware.py

@@ -313,7 +313,7 @@ def process_request(self, request):  # noqa
         try:
             project = Project.objects.get(slug=request.host_project_slug)
         except Project.DoesNotExist:
-            log.warning('No host_project_slug set on project')
+            log.debug("No host_project_slug set on project")
             return None
 
         # This is hacky because Django wants a module for the URLConf,

readthedocs/proxito/views/utils.py

@@ -1,6 +1,6 @@
 import os
-import structlog
 
+import structlog
 from django.http import HttpResponse
 from django.shortcuts import get_object_or_404, render
 
@@ -60,8 +60,8 @@ def _get_project_data_from_request(
         if lang_slug and version_slug:
             filename = os.path.join(lang_slug, version_slug, filename)
             log.warning(
-                'URL looks like versioned on a single version project.'
-                'Changing filename to match.',
+                "URL looks like versioned on a single version project. "
+                "Changing filename to match.",
                 filename=filename,
             )
             lang_slug = version_slug = None