potiuk commented on a change in pull request #16170:
URL: https://github.com/apache/airflow/pull/16170#discussion_r642138014
##########
File path: scripts/in_container/prod/entrypoint_prod.sh
##########
@@ -311,6 +311,16 @@ if [[ -n "${_AIRFLOW_WWW_USER_CREATE=}" ]] ; then
create_www_user
fi
+if [[ -n "${_PIP_ADDITIONAL_REQUIREMENTS=}" ]] ; then
+ echo "!!! WARNING !!! installing requirements with
_PIP_ADDITIONAL_REQUIREMENTS is for testing only!!!!"
+ echo
+ echo "Installing dependencies this way has serious performance
implications"
+ echo
+ echo "For production usage make sure to add dependencies to your image"
+ echo
Review comment:
```suggestion
>&2 echo "!!! WARNING !!! installing requirements with
_PIP_ADDITIONAL_REQUIREMENTS is for testing only!!!!"
>&2 echo
>&2 echo "Installing dependencies this way has serious performance
implications"
>&2 echo
>&2 echo "For production usage make sure to add dependencies to your
image"
>&2 echo
```
--
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]
