Merge pull request #6503 from stsewd/dont-pass-build-when-sync-repo

Don't pass build to environment when doing a sync

Author: stsewd

readthedocs/doc_builder/environments.py

@@ -481,7 +481,7 @@ def run_command_class(
                 log.warning(
                     LOG_TEMPLATE,
                     {
-                        'project': self.project.slug,
+                        'project': self.project.slug if self.project else '',
                         'version': 'latest',
                         'msg': msg,
                     }
@@ -570,8 +570,8 @@ def __exit__(self, exc_type, exc_value, tb):
         log.info(
             LOG_TEMPLATE,
             {
-                'project': self.project.slug,
-                'version': self.version.slug,
+                'project': self.project.slug if self.project else '',
+                'version': self.version.slug if self.version else '',
                 'msg': 'Build finished',
             }
         )
@@ -614,9 +614,9 @@ def handle_exception(self, exc_type, exc_value, _):
                 extra={
                     'stack': True,
                     'tags': {
-                        'build': self.build.get('id'),
-                        'project': self.project.slug,
-                        'version': self.version.slug,
+                        'build': self.build.get('id') if self.build else '',
+                        'project': self.project.slug if self.project else '',
+                        'version': self.version.slug if self.version else '',
                     },
                 },
             )

readthedocs/projects/tasks.py

@@ -243,7 +243,6 @@ def run(self, version_pk):  # pylint: disable=arguments-differ
             environment = LocalBuildEnvironment(
                 project=self.project,
                 version=self.version,
-                build=self.build,
                 record=False,
                 update_on_success=False,
             )