You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/07/12 11:36:30 UTC

[GitHub] [airflow] eladkal commented on a diff in pull request #24576: doc: reload pods when using the same DAG tag

eladkal commented on code in PR #24576:
URL: https://github.com/apache/airflow/pull/24576#discussion_r918865045


##########
docs/helm-chart/manage-dags-files.rst:
##########
@@ -71,12 +71,19 @@ Finally, update the Airflow pods with that image:
 
 If you are deploying an image with a constant tag, you need to make sure that the image is pulled every time.
 
+.. warning::
+
+    Using constant tag should be used only for testing/development purpose. It is a bad practice to use the same tag as you'll lose the history of your code.
+
 .. code-block:: bash
 
     helm upgrade --install airflow apache-airflow/airflow \
       --set images.airflow.repository=my-company/airflow \
       --set images.airflow.tag=8a0da78 \
-      --set images.airflow.pullPolicy=Always
+      --set images.airflow.pullPolicy=Always \
+      --set airflowPodAnnotations.random=r$(uuidgen)
+      

Review Comment:
   ```suggestion
   ```
   Remove whitespace to fix static tests



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