SEPURI-SAI-KRISHNA commented on code in PR #74171: URL: https://github.com/apache/airflow/pull/74171#discussion_r4176057812
########## providers/amazon/docs/changelog.rst: ########## @@ -26,6 +26,14 @@ Changelog --------- +.. warning:: + ``EcsRunTaskOperator`` and ``BatchOperator`` now pass ``verify`` and ``botocore_config`` to the + CloudWatch client that streams task logs, which previously used boto3 defaults. A deployment that + sets an aggressive ``botocore_config`` timeout for the task client will now apply it to log + streaming as well, and one that points ``verify`` at a private CA bundle will now use it there + too. ``EcsRunTaskOperator`` already honoured both settings when reading the final log line, so + this makes the two log paths agree. Review Comment: Leaving it for now as you suggest. The case for keeping it is that a tight `botocore_config` read timeout now applies to log streaming where it previously did not, so it can change behaviour for an existing deployment without any action on their part. If that is too small to warrant an entry, I will drop it. -- 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]
