Logging: reduce verbosity (readthedocs#9107)

geeksambhu · Apr 13, 2022 · 853f3fc · 853f3fc
1 parent ccdad23
commit 853f3fc
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/readthedocs/doc_builder/environments.py b/readthedocs/doc_builder/environments.py
@@ -646,7 +646,10 @@ def __exit__(self, exc_type, exc_value, tb):
                 container_id=self.container_id,
             )
         except DockerAPIError:
-            log.exception(
+            # Logging this as warning because it usually happens due memory
+            # limit or build timeout. In those cases, the container is not
+            # running and can't be killed
+            log.warning(
                 'Unable to kill container.',
                 container_id=self.container_id,
             )

diff --git a/readthedocs/projects/tasks/builds.py b/readthedocs/projects/tasks/builds.py
@@ -136,7 +136,6 @@ def on_failure(self, exc, task_id, args, kwargs, einfo):
         if isinstance(exc, RepositoryError):
             log.warning(
                 'There was an error with the repository.',
-                exc_info=True,
             )
         elif isinstance(exc, SyncRepositoryLocked):
             log.warning(