jedcunningham commented on a change in pull request #19453:
URL: https://github.com/apache/airflow/pull/19453#discussion_r746797884



##########
File path: docs/apache-airflow/installation/upgrading.rst
##########
@@ -0,0 +1,94 @@
+ .. Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+ ..   http://www.apache.org/licenses/LICENSE-2.0
+
+ .. Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+
+Upgrading Airflow to a newer version
+------------------------------------
+
+Why you need to upgrade
+=======================
+
+Newer Airflow versions can contain Database migrations so it is recommended 
that you run
+``airflow db upgrade`` to Upgrade your Database with the schema changes in the 
Airflow version
+you are upgrading to.
+
+When you need to upgrade
+========================
+
+If you have a custom deployment based on virtualenv or Docker Containers, you 
usually need to run
+the DB upgrade manually as part of the upgrade process.
+
+In some cases the upgrade happens automatically - it depends if in your 
deployment, the upgrade is
+built-in as post-install action. For example when you are using 
:doc:`helm-chart:index` with
+post-upgrade hooks enabled, the database upgrade happens automatically right 
after the new software
+is installed. Similarly all Airflow-As-A-Service solutions perform the upgrade 
automatically for you,
+when you choose to upgrade airflow via their UI

Review comment:
       ```suggestion
   when you choose to upgrade airflow via their UI.
   ```

##########
File path: airflow/www/templates/airflow/dags.html
##########
@@ -57,7 +57,8 @@
       Airflow found incompatible data in the <code>{{ original_table_name 
}}</code> table in the
       metadatabase, and has moved them to <code>{{ moved_table_name }}</code> 
during the database migration
       to upgrade. Please inspect the moved data to decide whether you need to 
keep them, and manually drop
-      the <code>{{ moved_table_name }}</code> table to dismiss this warning.
+      the <code>{{ moved_table_name }}</code> table to dismiss this warning. 
Read more about it
+      in <a href={{ get_docs_url("installing/upgrading.html") 
}}><b>Upgrading</b></a>

Review comment:
       ```suggestion
         in <a href={{ get_docs_url("installing/upgrading.html") 
}}><b>Upgrading</b></a>.
   ```




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


Reply via email to