potiuk commented on PR #32257: URL: https://github.com/apache/airflow/pull/32257#issuecomment-1612942463
I would be rather against that at least not until someone works on upgrading the mssql suport for ARM image. There are (were) various problems with MSSQL mostly related to not supporting all architectures and operating system - including the fact that for now we are excluding mssql-client from installation on ARM architecture https://github.com/apache/airflow/blob/main/scripts/docker/install_pipx_tools.sh#L27 This means that users of the ARM image will not be able to connect to the mssql database using client. Possibly this issu has been solved - but fixing it should be the first step. Do you have access to ARM/M1 to test it and make a fix to not exclude mssql-cli from installation and make such PR @manuel-lang ? Should be as easy as running `breeze prod-image build` after making the change to exclude ARM and running pre-commit (to regenerate scripts embedded in Dockerfiles). -- 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]
