o-nikolas commented on PR #35713: URL: https://github.com/apache/airflow/pull/35713#issuecomment-1817139400
> Is the added config of `cloudwatch_task_handler_json_serializer` relevant for watchtower<3 ? If not we should also edit the description of the setting with note about it to avoid confusion Good question @eladkal! I did check before submitting the PR that the 2.0.1 version of the Watchtower code to ensure that parameter is accepted and it looks like it is: https://github.com/kislyuk/watchtower/blob/v2.0.1/watchtower/__init__.py#L185 The default serializer we provide was a fallback for the one WT uses anyways, and providing a new one should still work just as intended with 2.0.1. So I think it's all good. But maybe @cBiscuitSurprise can confirm just in case. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
