wolfdn commented on PR #33914: URL: https://github.com/apache/airflow/pull/33914#issuecomment-1700435399
I just noticed that there are some unit tests that check if the raised Exception of the Docker operator contains the container logs. Before I'm adapting / removing those checks: Is there actually a reason why we want to have the container logs attached to the raised exception? I mean the logs of the Docker container are displayed anyways in Airflow (even before the exception is raised). I currently only see the downside that the logs are printed 3 times in Airflow if we also attach them to the exception. But maybe there is a good reason why we want/need this? -- 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]
