[GitHub] [airflow] potiuk commented on a change in pull request #19993: Update upgrading.rst with detailed code example of how to resolve post-upgrade warning

[GitBox <gi...@apache.org>]

potiuk commented on a change in pull request #19993:
URL: https://github.com/apache/airflow/pull/19993#discussion_r762991737



##########
File path: docs/apache-airflow/installation/upgrading.rst
##########
@@ -79,6 +79,20 @@ table or rename it or move it to another database using those tools. If you don'
 can use the ``airflow db shell`` command - this will drop you in the db shell tool for your database and you
 will be able to both inspect and delete the table.
 
+Alternatively, you can drop the table by following the steps below:
+
+
+1. Exec into any of the Airflow pods - webserver or scheduler: ``kubectl exec -it <your-webserver-pod> python``

Review comment:
       I'd rather give some more examples - kubernetes, docker, docker-compose and tell them this is an example and that they should find out what they should do in their environment.
   
    I think our assumption is that this documentation is used by  users who had never done low-level DB operations with Airflow DB, and they likely have no idea how to get to their containers. Giving them at least a handy example they could follow and at least the starting point that will allow them to ask the right google question to dig deeper is useful.




-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org