potiuk opened a new pull request #11174:
URL: https://github.com/apache/airflow/pull/11174


   **NOTE TO REVIEWER** - please review only the last commit - the other is 
part of #11173
   
   This is the second step of making the Production Docker Image more
   corporate-environment friendly, by making MySQL client installation
   optional. Instaling MySQL Client on Debian requires to reach out
   to oracle deb repositories which might not be approved by security
   teams when you build the images. Also not everyone needs MySQL
   client or might want to install their own MySQL client or MariaDB
   client - from their own repositories.
   
   This change makes the installation step separated out to
   script (with prod/dev installation option). The prod/dev separation
   is needed because MySQL needs to be installed with dev libraries
   in the "Build" segment of the image (requiring build essentials
   etc.) but in "Final" segment of the image only runtime libraries
   are needed.
   
   Part of #11171
   
   Depends on #11173.
   
   
   **^ Add meaningful description above**
   
   Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
 for more information.
   In case of fundamental code change, Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals))
 is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in 
[UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md).
   


----------------------------------------------------------------
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]


Reply via email to