nullhack edited a comment on pull request #9464: URL: https://github.com/apache/airflow/pull/9464#issuecomment-703541914
Thank you, yes you're right. They're failing because the test assume a binary string `b'container log'` One of the modifications of this PR is changing this to a decoded string instead (to have same output type as bash operator and make It easier to use the xcom results from other operators). At this point I have three options 1. Change the behavior of the tests to the new one (can introduce breaking changes, as the current behavior is binary) 2. Introduce a new flag argument for the docker operator with default binary, but modifiable to include the new behavior 3. Remove the new behavior Which one should I go for? ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
