Taragolis commented on code in PR #36716:
URL: https://github.com/apache/airflow/pull/36716#discussion_r1447825517
##########
docker_tests/test_prod_image.py:
##########
@@ -158,6 +158,7 @@ def test_pip_dependencies_conflict(self):
"grpc": ["grpc", "google.auth", "google_auth_httplib2"],
"hashicorp": ["hvac"],
"ldap": ["ldap"],
+ "mysql": ["MySQLdb", *(["mysql"] if bool(find_spec("mysql")) else [])],
Review Comment:
Yeah I think it would be the best option, rather than think provide
workaround how to properly remove MySQL libraries and replace them by MariaDB.
Otherwise we would have multiple issues everywhere: Github / Slack / SO.
Seems like all holidays is over and users started to upgrade their Airflow
instances 🤣
--
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: commits-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
