You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/11/30 22:26:11 UTC

[GitHub] [airflow] o-nikolas commented on a change in pull request #19907: Move to watchtower 2.0.1 - Fixes Watchtower crash when logging empty line

o-nikolas commented on a change in pull request #19907:
URL: https://github.com/apache/airflow/pull/19907#discussion_r759708573



##########
File path: airflow/providers/amazon/aws/log/cloudwatch_task_handler.py
##########
@@ -81,7 +81,7 @@ def set_context(self, ti):
         self.handler = watchtower.CloudWatchLogHandler(
             log_group=self.log_group,
             stream_name=self._render_filename(ti, ti.try_number),
-            boto3_session=self.hook.get_session(self.region_name),
+            boto3_client=self.hook.get_conn(),

Review comment:
       Watchtower now expects a Boto3 client instead of a session, this was changed in this commit: https://github.com/o-nikolas/watchtower/commit/a6232505d5aba2f5d2a67641e30bfbaa5874a0fa




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org