Logs: add build_id attribute to set_builder_scale_in_protection (#12504)

This will help us to know what builds weren't unable to set AWS scale-in
protection. It could give us some insights about why these builds were
killed.

Author: humitos

readthedocs/projects/tasks/builds.py

@@ -430,6 +430,7 @@ def before_start(self, task_id, args, kwargs):
         # once we don't need to rely on `self.data.project`.
         if self.data.project.has_feature(Feature.SCALE_IN_PROTECTION):
             set_builder_scale_in_protection.delay(
+                build_id=self.data.build_pk,
                 builder=socket.gethostname(),
                 protected_from_scale_in=True,
             )
@@ -783,6 +784,7 @@ def after_return(self, status, retval, task_id, args, kwargs, einfo):
         # Disable scale-in protection on this instance
         if self.data.project.has_feature(Feature.SCALE_IN_PROTECTION):
             set_builder_scale_in_protection.delay(
+                build_id=self.data.build_pk,
                 builder=socket.gethostname(),
                 protected_from_scale_in=False,
             )

readthedocs/projects/tasks/utils.py

@@ -192,14 +192,15 @@ def send_external_build_status(version_type, build_pk, commit, status):
 
 
 @app.task(queue="web")
-def set_builder_scale_in_protection(builder, protected_from_scale_in):
+def set_builder_scale_in_protection(build_id, builder, protected_from_scale_in):
     """
     Set scale-in protection on this builder ``builder``.
 
     This way, AWS will not scale-in this builder while it's building the documentation.
     This is pretty useful for long running tasks.
     """
     structlog.contextvars.bind_contextvars(
+        build_id=build_id,
         builder=builder,
         protected_from_scale_in=protected_from_scale_in,
     )