potiuk commented on pull request #18037: URL: https://github.com/apache/airflow/pull/18037#issuecomment-913150704
> Maybe we should also add a marker to skip Python 2 tests if `python2` is not available? THis can be dealed separately though. I think that's not good idea. If we did that before, we would have never known that python slim-buster suddenly without any warning dropped python2 from their image and we would not know Airflow PROD image suddenly lost the capacity of running Python 2 via PythonVirtualenvOperator. For now this is our official recommendation if you still want to use Python2 when migrating to Python2 (and for some people it is still important). So in a way the test actually tests what we promised. We should simply remove both test and python2 when we drop this "Python2" fallback (likely Airflow 3). -- 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]
