Adaverse commented on code in PR #32810:
URL: https://github.com/apache/airflow/pull/32810#discussion_r1278133857
##########
docs/docker-stack/entrypoint.rst:
##########
@@ -315,8 +315,8 @@ either as maintenance operations on the database or should
be embedded in the cu
Upgrading Airflow DB
....................
-If you set :envvar:`_AIRFLOW_DB_UPGRADE` variable to a non-empty value, the
entrypoint will run
-the ``airflow db upgrade`` command right after verifying the connection. You
can also use this
+If you set :envvar:`_AIRFLOW_DB_MIGRATE` variable to a non-empty value, the
entrypoint will run
Review Comment:
Renamed the env var as well for consistency!
##########
scripts/docker/entrypoint_prod.sh:
##########
@@ -283,8 +283,8 @@ if [[ "${CONNECTION_CHECK_MAX_COUNT}" -gt "0" ]]; then
wait_for_airflow_db
fi
-if [[ -n "${_AIRFLOW_DB_UPGRADE=}" ]] ; then
- upgrade_db
+if [[ -n "${_AIRFLOW_DB_UPGRADE=}" ]] || [[ -n "${_AIRFLOW_DB_MIGRATE=}" ]] ;
then
Review Comment:
Made this change so that its backward compatible
--
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]
